# |
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 |