summaryrefslogtreecommitdiffstats
path: root/tests/ConcertTest.php
Commit message (Collapse)AuthorAgeFilesLines
* Clean up, fix and rename db tables.Harald Eilertsen2021-09-161-2/+2
| | | | | | | | | | | | | | | | | This patch got a bit more involved than what was originally planned, but since we're messing with the tables I decided to do it all right away. - Moves the constraint definition to the CREATE TABLE statement for the concerts table. This replaces the existing KEY definition that it had. - Make sure the venues table is created before the concerts table so that the above mentioned constraint definition works. - Rename the tables. Use the wpdb-prefix and make the name a bit prettier. This caused some changes in the Concert and Venue classes, and for slightly silly reasons some test classes. The code actually turned out better (for the most part), but some refactoring can still be done. The column names remains unchanged for now.
* Fix ConcertTest::testFetchConcertByVenueAndDate.Harald Eilertsen2021-09-161-1/+1
| | | | | | | | Since the date column has changed types, so has the representation in the concert class. We should really change the property in the Concert class be a proper DateTime object.
* Throw instead of log error from Concert class.Harald Eilertsen2021-09-061-4/+4
| | | | | Should get rid of most of the annoying output during testing, and allow moving error handling and logging to the presentation layer.
* Use find_concerts to check for duplicates on create.Harald Eilertsen2021-09-061-0/+16
| | | | This also adds a number of new filters to find_concerts.
* test: concertlogs table is gone, no need to clear itHarald Eilertsen2021-09-051-1/+0
|
* Fix saving and fetching roles from Concerts table.Harald Eilertsen2021-09-011-19/+4
|
* Begin move roles and status field to concerts table.Harald Eilertsen2021-06-271-0/+25
|
* Refactor ConcertTest some more.Harald Eilertsen2021-06-271-58/+25
| | | | | Create more of the concerts used by tests into the wpSetupBeforeClass hook.
* Refactor ConcertTests and some cleanupHarald Eilertsen2021-06-271-68/+75
| | | | | | | | | | | The commit changes the way we populate the database for the tests by creating more entries up front. This reduces the amount of duplicated code between the tests, but also introduce some challenges. As modifications to the database done in the wpSetUpBeforeClass hook are not cleaned up automatically by the WP_PHPUnit framework, we also have to add a wpTearDownAfterClass hook so anything we set up in this class does not disturb any other tests in other classes.
* Begin move roles and status field to concerts table.Harald Eilertsen2021-06-141-0/+24
| | | | There's no need to have a separate table (concertlogs) for these fields.
* Streamline Concert api.Harald Eilertsen2021-06-121-7/+7
| | | | | Reduce to one find_concerts function taking a filter to limit the selection.
* Make it possible to list all concerts.Harald Eilertsen2021-05-101-0/+23
|
* Method to find concerts at given venueHarald Eilertsen2021-05-091-0/+40
|
* Method to find concerts for a given city.Harald Eilertsen2021-05-091-0/+41
|
* Include info from venue in concerts.Harald Eilertsen2021-05-091-1/+20
| | | | | | | | | | | | | | | | | | | | This makes the concert a full object containing all relevant info, while we can still segment the data in the db. Instead of this: $concert = GiglogAdmin_Concert::get($concert_id); $venue = GiglogAdmin_Venue::get($concert->venue()); echo "{$concert->name()} @ {$venue->name()} : {$concert->cdate()}" You can now do: $concert = GiglogAdmin_Concert::get($concert_id); echo "{$concert->name()} @ {$concert->venue()->name()} : {$concert->cdate()}" And yeah, renamed Concert::find_cid() to Concert::get() and changed it's semantics somewhat. It now either returns the given concert if it exists, or NULL if it does not. Simpler function; simpler to use.
* Added an uppercase in sql for venue and concert duplicate checkAndreaChirulescu2021-04-231-0/+22
| | | | Added test to create duplicate concert with varied cases in string
* Concert::create return NULL if concert already exist.Harald Eilertsen2021-04-221-1/+1
| | | | | | We probably need some better error handling here. There's a myriad of reasons why this call could fail, and we might need to communicate the failure reason somewhere.
* Add some tests for Concert class.Harald Eilertsen2021-04-221-0/+49