Gj

I still encounter this problem.

You commented out the first occurrence but not the second occurrence.

test.expect(default2 > now2 > default > now, msg)
TypeError: '>' not supported between instances of 'NoneType' and 'datetime.datetime'

Where so? in the samplet tests, I guess? Then you're(were) using an outdated version of the sample tests: I commented out that test in both the sample tests and the full test suite ages ago.

I want to continue this discussion.

I also kept getting the same error:

    test.expect(default2 > now2 > default > now, msg)
TypeError: '>' not supported between instances of 'NoneType' and 'datetime.datetime'

And after expanding the further comments, it is revealed that the test contains:

field2 = DateTimeField(auto_now=True, blank=True)
default = field2.default

This feels out of scope, since DateTimeField shouldn't be accessed outside a class. I had to put in extra hacks in my solution to get this to work.

Informational question only: Will descriptors work in place of metaclasses here? I understand the purpose of the kata is to learn metaclasses.

removed => isinstance is now allowed

.

handled, testing only at Model instantiation time. Oh, I forgot to talk about default taking precedence over auto_now. I'll add that in the next fork.

suppressed the meaningless requirement (and updated the description)

Thanks again for the feedback and the time you spent on this! I did have to check my validation, probably have to refactor it later.

Tests passed! Thanks for modifying the tests! Really helped a lot!

Thanks for looking into it and spending the time! I really appreciate it. I will look into to checking your comment on the DatetimeField.

I updated the tests so that the error message tells what fields are incorrect. Please reset the trainer and reload the page (don't forget to copy your current code first).

I just tried your code: you're actually looking at the wrong field (dunno why). Your code fails on instances where the DateTimeField is wrong, not the IntegerField.