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] template: PhantomJS Test Sample #215

Merged
merged 2 commits into from Nov 7, 2016

Conversation

lasley
Copy link
Contributor

@lasley lasley commented Aug 18, 2016

Milestone (Odoo version)

  • 9.0

Module(s)

  • Template

Fixes / new features

  • Adds an example for QUnit test suite injection via QWeb
  • Adds PhantomJS test example

@pedrobaeza
Copy link
Member

I think we should have branches 8.0, 9.0 of MT to accommodate all of these changes to all versions. Can you please bring the topic to mailing list?

About this, isn't needed a Python file to launch the test?

@lasley
Copy link
Contributor Author

lasley commented Aug 18, 2016

@pedrobaeza you are correct that a Python file is also required - we already had that part in our template for some reason.

Writing the email now

odoo.define_section('module_name', ['module_name.ExportedObject'], function(test) {
"use strict";

test('It should demonstrate a PhantomJS test',
Copy link
Member

Choose a reason for hiding this comment

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

Please specify that this is for backend tests.

function(assert, ExportedObject) {
var expect = 'Expected Return',
result = new ExportedObject();
assertStrictEqual(
Copy link
Member

Choose a reason for hiding this comment

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

this must be assert.assertStrictEqual, right?

* Change description to include that it is for web/backend
* Fix assertion on no object
@lasley lasley force-pushed the feature/9.0/add-phantom-example branch from 1ed89cf to 19a5b41 Compare November 3, 2016 15:37
@lasley
Copy link
Contributor Author

lasley commented Nov 3, 2016

Thanks @yajo - updated

@oscarolar
Copy link

👍

@pedrobaeza pedrobaeza merged commit 3f474e2 into OCA:master Nov 7, 2016
@lasley lasley deleted the feature/9.0/add-phantom-example branch November 7, 2016 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants