Skip to content

[ADD] inhibit warnings from acl tests - #427

Merged
pedrobaeza merged 1 commit into
OCA:8.0from
hbrunn:8.0-base_suspend_security-mute_logger
Aug 5, 2016
Merged

[ADD] inhibit warnings from acl tests#427
pedrobaeza merged 1 commit into
OCA:8.0from
hbrunn:8.0-base_suspend_security-mute_logger

Conversation

@hbrunn

@hbrunn hbrunn commented May 18, 2016

Copy link
Copy Markdown
Member

No description provided.

@hbrunn hbrunn added this to the 8.0 milestone May 18, 2016
@coveralls

coveralls commented May 18, 2016

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 57.265% when pulling ce12cab on hbrunn:8.0-base_suspend_security-mute_logger into aae74f0 on OCA:8.0.

@pedrobaeza

Copy link
Copy Markdown
Member

Isn't better to inhibit the functionality when tests are enabled and you're testing other modules?

@hbrunn

hbrunn commented May 18, 2016

Copy link
Copy Markdown
Member Author

yes, but we have no influence on that: https://github.com/OCA/OCB/blob/8.0/openerp/addons/base/ir/ir_model.py#L767

@pedrobaeza

Copy link
Copy Markdown
Member

OK, I see. Then 👍

@moylop260

Copy link
Copy Markdown
Contributor

👍

@pedrobaeza
pedrobaeza merged commit b717070 into OCA:8.0 Aug 5, 2016
@gurneyalex

Copy link
Copy Markdown
Member

@hbrunn @pedrobaeza this merge broke the 8.0 build.

@pedrobaeza

Copy link
Copy Markdown
Member

It was green in its moment, so I don't know what is happening. Do you have any clue?

@gurneyalex

Copy link
Copy Markdown
Member

I don't know, I just noticed the travis builds turned red on the build of the merge. Can be unrelated to this PR and related to #505 (database_cleanup breaking the build on 9.0), since the errors seem to be related to that module.

@gurneyalex

Copy link
Copy Markdown
Member

see also #507

SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (11.0)
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.

5 participants