Commit Graph

5 Commits

Author SHA1 Message Date
Jairo Llopis
8b5c4f3fb0 [FIX] base_report_to_printer: Stateless tests (#122)
If the test was run in a database with the `account` module installed, they were failing for 2 reasons:

- It was searching for the first report it could find, with the first 5 records to report. This made it load the `account.report_agedpartnerbalance` report, which caused a fake `TypeError: 'NoneType' object has no attribute '__getitem__'` error.
- It was running tests without loading the full module graph, thus not getting the default value for the new required `invoice_warn` field.

Now tests are run in `post_install` mode to load full module graph, and they use stateless data.
2018-01-11 14:19:47 +01:00
Sylvain GARANCHER
a074ab4979 [FIX] Use an HttpCase test class to avoid locking the registry for HTTP threads loading during tests that need wkhtmltopdf 2017-08-26 09:19:20 +02:00
Sylvain Garancher
3f3d06a004 [10.0][MIG] base_report_to_printer (#68)
* Set api.multi for action called as `object` on view

* Merge syleam printers module into base_report_to_printer (#60)

* [IMP] Updated unit tests

* [FIX] Fixed renamed attributes

* [FIX] Remove deleted fields

* [IMP] Add printing.server and printing.job models

* [IMP] Allow to cancel all jobs, enable, and disable printers

* [IMP] Split the cups part of print_document into a new print_file method

* [IMP] Updated cron job to run the action_update_jobs method

* [ADD] Add a migration script to create a printing server from configuration

* [MIG] Migrate base_report_to_printer to v10.0

Removed deprecated methods on printing.printer (replaced by methods on
        printing.server)

* [IMP] Add wkhtmltopdf in travis configuration file

* [FIX] base_report_to_printer: Fix Update Job Cron
* Fix API issue with Update Job Cron
** Forward Port from 9.0

* [FIX] Fixed the res.users view

The string attribute should not be used as a selector, because it is
translatable.

* [FIX] Fixed the print_document method of report

The new API migration was made to @api.multi because of the "cr, uid,
ids" signature, but "ids" was the ids of the records to print here, not
the report's ids.
Also, the new API version of "get_pdf" get directly the ids of the
records to print in the standard module, not a recordset.

* [FIX] UI is now (un)blocked only when using qweb-pdf reports in standard addons
2017-04-05 09:06:16 -07:00
Dave Lasley
10bfbe6167 [FIX] base_report_to_printer: Remove execute bit 2016-07-12 19:19:22 -07:00
Dave Lasley
8e896af6a9 [IMP] base_report_to_printer: Add test coverage
* Add test coverage
* Minor touch ups in wizard
* Prefer UserError to Warning
2016-07-07 18:14:28 -07:00