Log message #4101885

# At Username Text
# Dec 7th 2017, 14:14 neon1024 Not about where to put a test?
# Dec 7th 2017, 14:14 birdy247 But point taken
# Dec 7th 2017, 14:14 birdy247 I was asking a refactor question
# Dec 7th 2017, 14:13 birdy247 It does exist :slightly_smiling_face:
# Dec 7th 2017, 14:13 neon1024 You cannot refactor code which doesn’t exist
# Dec 7th 2017, 14:13 neon1024 If it works, super, if you can refactor it to make it less crap, great!
# Dec 7th 2017, 14:12 neon1024 So just write what you want to write and crack on
# Dec 7th 2017, 14:12 neon1024 You can try all you want to get stuff right, but in the end it’ll be garbage in a week anyway
# Dec 7th 2017, 14:12 neon1024 Which is why pre-optimisation is the devils work
# Dec 7th 2017, 14:12 neon1024 Procrastinating about best practise delays development
# Dec 7th 2017, 14:12 neon1024 Getting stuff working moves your project on
# Dec 7th 2017, 14:12 neon1024 As soon as your save your code it’s legacy
# Dec 7th 2017, 14:11 neon1024 Or if someone tells you there is, next week it’ll be legacy and the next thing will come along
# Dec 7th 2017, 14:11 neon1024 Like most things in life there isn’t a best way
# Dec 7th 2017, 14:11 neon1024 You’re pre-optimising
# Dec 7th 2017, 14:11 neon1024 I disagree
# Dec 7th 2017, 14:10 birdy247 Short term loss long term gain, would rather get things right to save time further down the line
# Dec 7th 2017, 14:10 neon1024 See what works for you
# Dec 7th 2017, 14:10 neon1024 Just experiment and have a play about
# Dec 7th 2017, 14:10 neon1024 You seem to need someone to sign off all your choices before you make them
# Dec 7th 2017, 14:09 neon1024 I wonder how you get any work done
# Dec 7th 2017, 14:09 neon1024 You doubt yourself a lot
# Dec 7th 2017, 14:09 birdy247 its quite involved
# Dec 7th 2017, 14:09 birdy247 so is this a sign this method is in the wrong place?
# Dec 7th 2017, 14:09 neon1024 They often don’t contain sufficient logic to warrant testing
# Dec 7th 2017, 14:09 neon1024 In most cases I wouldn’t bother
# Dec 7th 2017, 14:08 birdy247 like how would you test the getters and setters work as expected?
# Dec 7th 2017, 14:05 birdy247 so would I create a test in the model?
# Dec 7th 2017, 14:04 birdy247 into the entity
# Dec 7th 2017, 14:04 birdy247 We have added a method called getPaymentDate
# Dec 7th 2017, 14:04 neon1024 There is nothing to test, which is why bake doesn’t write any tests
# Dec 7th 2017, 14:04 birdy247 dont need any tests?
# Dec 7th 2017, 14:03 neon1024 Plain old php object
# Dec 7th 2017, 14:03 birdy247 POPO ?
# Dec 7th 2017, 14:03 johnwayne How can I call AppController from another class in another folder ex. src/Controller/Utils/lists.php (List class) and from there I want to call method from AppController
# Dec 7th 2017, 14:03 birdy247 but presumably just making a new Entity folder and additing one myself is the way to test these methods
# Dec 7th 2017, 14:03 neon1024 Well they’re POPO’s, they don’t need any
# Dec 7th 2017, 14:03 birdy247 I notice that cake doesnt bake any tests for entities
# Dec 7th 2017, 14:02 birdy247 @neon1024 thanks
# Dec 7th 2017, 14:02 neon1024 No @birdy247
# Dec 7th 2017, 13:58 saeideng maybe needs to fix in debugKit