don't
# suitescriptc
creece
12/14/2018, 7:39 PMdon't
s
screnshaw
12/14/2018, 7:41 PMHa, Why not? Being able to run all of our unit tests for Updates/Upgrades/etc would be huge.
c
creece
12/14/2018, 7:41 PMYeah its sound great in theory but its difficult to do. You have to separate all of your business logic from actual netsuite calls. For example if you are calculating dates, you'd need a method that takes in data and doesn't do netsuite calls.
creece
12/14/2018, 7:43 PMor encapsulate all of your netsuite calls inside other methods and mock those methods for return values that you want.. just don't test netsuite itself test your business logic only
creece
12/14/2018, 7:43 PMYou can use mocha/should or whatever node based framework you want after that to run the tests
s
screnshaw
12/14/2018, 7:45 PMYeah, and that's kind of what we want to do. Test our business logic
screnshaw
12/14/2018, 7:45 PMAlthough being able to query the db to test for expected results would be nice
c
creece
12/14/2018, 7:45 PMyou'll have to mock your data
creece
12/14/2018, 7:46 PMyou shouldn't be testing db results as you aren't testing netsuite. You're testing your business logic.
s
screnshaw
12/14/2018, 7:47 PMSounds like a monstrous headache to do, but since it is slow due to the end of the year, there's no better time to attempt it.
c
creece
12/14/2018, 7:47 PMfor example if you're calculating a custom value to set on a field on a record, you need to test the actual calculation not that it was set on a record. Once you set it on a record thats netsuite and is out of your control. You can say that your value(s) are correct though via unit tests
👍🏼 1
s
screnshaw
12/14/2018, 7:47 PMIt'd be nice to test other logic as well based on a configuration of things
c
creece
12/14/2018, 7:48 PMyou just gotta separate the netsuite calls from the actual business logic.. so instead of having a method that uses suitescript api to get the values and do the calculation, you'd get the suitescript api values and pass them into a method that is independent of netsuite api calls
creece
12/14/2018, 7:49 PMor you can use something like that NFT or your own methods that wrap native netsuite api methods so you can mock them
s
screnshaw
12/14/2018, 7:50 PMThat'd be a lot of changes. Maybe something going forward we attempt with small things at first
screnshaw
12/14/2018, 7:50 PMIs that the fasttrack toolkit?
c
creece
12/14/2018, 7:50 PMyeah
creece
12/14/2018, 7:52 PMits an API wrapper so you can mock stuff
creece
12/14/2018, 7:52 PMi don't recommend it but im sure someone will
s
screnshaw
12/14/2018, 7:54 PMHave you seen the help docs for
Project Sample: SuiteScript 2.0 Unit Testingc
creece
12/14/2018, 8:01 PMi dont believe so what does it suggest
creece
12/14/2018, 8:24 PMthats pretty interesting actually
creece
12/14/2018, 8:26 PMthey are just mocking everything like i was suggesting
s
screnshaw
12/14/2018, 8:27 PMYeah, I thought so too.
screnshaw
12/14/2018, 8:27 PMsorry, got pulled away on a meeting but it seems that you opened it up
h
Hubert
12/14/2018, 8:32 PMc
creece
12/14/2018, 8:34 PMThat framework looks like its never testing anything
creece
12/14/2018, 8:34 PMyou mock everything so your results will always match what you'd expect
creece
12/14/2018, 8:35 PMi could be totally wrong and probably am
s
stalbert
12/14/2018, 11:25 PMmeh, I use jest and NFT and while that makes unit testing easier in some cases NetSuite unit testing is still more challenging than the real world.