Ticket #23 (closed enhancement: fixed)

Opened 6 years ago

Last modified 6 years ago

Multi-PK tests

Reported by: guest Owned by:
Priority: minor Milestone:
Component: tests Version:
Keywords: Cc: elixir-multipktest@…

Description

School's starting to wind down to I updated my Elixir trunk and took a look at the TODO; this looked bite-sized enough to take on. Attached it a patch that tests multiple PKs, albeit not extensively, and removes the TODO entry.

Also, should there perhaps be an elixir-tests component in Trac?

-- Adam

Attachments

multipk_test.patch (2.8 kB) - added by guest 6 years ago.
Update my patch, previous one had the docstring on the test_ function, which was showing up in nosetests.
selfref_test.patch (0.8 kB) - added by guest 6 years ago.
Additional patch that adds a test for self-referential relations.

Change History

Changed 6 years ago by guest

Update my patch, previous one had the docstring on the test_ function, which was showing up in nosetests.

Changed 6 years ago by guest

Additional patch that adds a test for self-referential relations.

Changed 6 years ago by ged

  • status changed from new to closed
  • resolution set to fixed
  • component changed from elixir-core to elixir-tests

I'm truly sorry but those tests already existed in other files. I left that TODO item because it wasn't completely done (we are missing the ones in the autoload case). And the test_selfref file was just a leftover of my latest test reorganisation. Sorry again. I've updated the TODO file and removed the file so that it hopefully doesn't happen again. And thanks a lot for the contributions anyway.

Note: See TracTickets for help on using tickets.