-
Notifications
You must be signed in to change notification settings - Fork 23
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
Issue 56 - Fix Caching issues with WhatsApp button #57
Conversation
48eca56
to
a8328e7
Compare
self.execute_upgrade_step(step) | ||
|
||
# Check | ||
self.assertTrue(JS_ID in js_tool.getResourceIds()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use assertIn
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it breaks tests on Plone 4.1
8fc03e6
to
9b0d200
Compare
@@ -7,12 +7,3 @@ | |||
</div> | |||
</tal:plugin> | |||
</div> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moving this code to a better place.
491dbeb
to
77fb912
Compare
suite = unittest.TestSuite() | ||
suite.addTests([ | ||
layered( | ||
robotsuite.RobotTestSuite(t, noncritical=['Expected Failure']), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rodfersou you forgot to add this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Issue 56 - Fix Caching issues with WhatsApp button
closes #56