Skip to content

Fixing unit tests#76

Open
pkamps wants to merge 16 commits into
ezsystems:masterfrom
pkamps:fixing_unit_test
Open

Fixing unit tests#76
pkamps wants to merge 16 commits into
ezsystems:masterfrom
pkamps:fixing_unit_test

Conversation

@pkamps

@pkamps pkamps commented Sep 19, 2012

Copy link
Copy Markdown
Contributor

Fixing fatal errors in unit tests. Fixing most failing tests - some tests still fail though

@bdunogier

Copy link
Copy Markdown
Contributor

Okay for me. Any objection, anyone ? @dpobel ? @paulborgermans ?

@dpobel

dpobel commented Sep 20, 2012

Copy link
Copy Markdown
Contributor

+1

@paulborgermans

Copy link
Copy Markdown
Contributor

Fine for me

@nfrp

nfrp commented Sep 20, 2012

Copy link
Copy Markdown
Contributor

Schön :)

@jeromegamez

Copy link
Copy Markdown
Contributor

+1, thanks @pkamps, I was just scratching my head during the setup of a testing environment :).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this and we are ready to go :)

@guillaumelecerf

Copy link
Copy Markdown
Contributor

Hi @pkamps, could you please rebase your patch serie ? Because some parts have already been fixed in HEAD.

@pkamps

pkamps commented Nov 6, 2013

Copy link
Copy Markdown
Contributor Author

My current work load won't allow me to do this in the next 3 weeks.

Jérôme Gamez and others added 5 commits October 24, 2014 13:33
The table 'ezfind_elevate_configuration' is already created in the setUp() method of the eZ Find Test Suite
…_sql_data

EZP-21779: Removed duplicate table creation
Implement EZP-21380: 'highlighting' as a SearchExtras attribute
Implement EZP-21404: Allow excluding object owner/author from search
Fix EZP-21331: fatal error due to invalid facet def passed to fetch function
@guillaumelecerf

Copy link
Copy Markdown
Contributor

@andrerom : some parts of this PR must be merged to fix the unit tests.
@pkamps : could you please rebase this one with the current HEAD ?

@pkamps

pkamps commented Oct 24, 2014

Copy link
Copy Markdown
Contributor Author

I rebased but I'm not sure if it's still a valid pull request.

@andrerom

Copy link
Copy Markdown
Contributor

@pkamps Thanks :)
@guillaumelecerf Up for running unit tests again with this one? (but ok, even if failing travis will provide some value, so should try to do that next week)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

9 participants