Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test cases for some weird issues with Event content type #407

Closed
wants to merge 1 commit into from

Conversation

@hvelarde
Copy link
Member Author

@idgserpro I tested both weird issues we are having in collective.cover and was not able to reproduce them here; I think we have a problem in our test suite setup and we would need some help.

@hvelarde
Copy link
Member Author

@idgserpro any feedback?

@idgserpro
Copy link

@hvelarde are you sure that the plone.api tests are run on the coredev of Plone 4.3? It's not in tests.cfg.

And the travis buildout is not working.

@idgserpro
Copy link

@hvelarde see #367

@hvelarde
Copy link
Member Author

@idgserpro I tested locally in my own machine and tests are passing; you can do the same.

@idgserpro
Copy link

idgserpro commented Aug 13, 2018

@hvelarde I run the Archetype test locally and it really passed

It looks like #364 has been fixed. About the dx problem I do not know much.

Do these two problems occur in cover?

@hvelarde
Copy link
Member Author

yes they do, and that's why I think we have a problem with the test fixture.

@@ -376,6 +376,46 @@ def test_create_at_with_title_in_request(self):

self.assertEqual(page.title, 'Test document')

@unittest.skipIf(HAS_PACONTENTYPES, 'Archetypes only')
def test_create_at_event(self):

Choose a reason for hiding this comment

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

@hvelarde why do not you add this test to the package? It will prevent #364 from occurring again.

Copy link
Member Author

Choose a reason for hiding this comment

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

Archetypes is going to die soon.


@unittest.skipUnless(HAS_PACONTENTYPES, 'Dexterity only')
def test_create_dx_event(self):
"""https://github.com/plone/plone.app.contenttypes/issues/465"""
Copy link

@idgserpro idgserpro Sep 6, 2018

Choose a reason for hiding this comment

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

@hvelarde what's the relationship between plone/plone.app.contenttypes#465 and plone.api? It seems to me more a problem of plone.app.contenttypes. What's purpose of this test?

@gbastien FYI.

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

Successfully merging this pull request may close these issues.

2 participants