~kotodama/charm-k8s-discourse:fix_unit_tests

Branch merges

Propose for merging

Merged into charm-k8s-discourse:master at revision 418f0952a76219225d17c14358679b09934b23f3

Tom Haddon: Approve on 2022-02-03

🤖 prod-jenkaas-is (community): Approve (continuous-integration) on 2022-02-03

Canonical IS Reviewers: Pending requested 2022-02-02

Diff: 305 lines (+22/-67)

11 files modified

dev/null (+0/-25)
tests/unit/fixtures/config_invalid_bad_throttle_mode.yaml (+0/-3)
tests/unit/fixtures/config_invalid_force_saml_no_saml_target.yaml (+0/-3)
tests/unit/fixtures/config_invalid_missing_cors.yaml (+0/-2)
tests/unit/fixtures/config_valid_complete.yaml (+0/-4)
tests/unit/fixtures/config_valid_no_saml_fingerprint.yaml (+0/-4)
tests/unit/fixtures/config_valid_no_saml_target_url.yaml (+0/-4)
tests/unit/fixtures/config_valid_no_tls.yaml (+0/-4)
tests/unit/fixtures/config_valid_with_tls.yaml (+0/-4)
tests/unit/fixtures/config_valid_without_tls.yaml (+0/-4)
tests/unit/test_charm.py (+22/-10)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

fcbab74... by Loïc Gomez on 2022-02-02

Fix unit tests since we removed some options / use juju relations

d1369de... by Tom Haddon on 2021-11-30

Update image location to the published location on dockerhub

Reviewed-on: https://code.launchpad.net/~mthaddon/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412584
Reviewed-by: Loïc Gomez <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

b98ef74... by Tom Haddon on 2021-11-30

Update image location to the published location on dockerhub

71f7b9d... by Barry Price on 2021-11-30

Remove extra precompile line

Reviewed-on: https://code.launchpad.net/~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412564
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

6c68be8... by Barry Price on 2021-11-30

Remove extra precompile line

54f81a9... by Barry Price on 2021-11-30

We can't tell from bundler's return code whether it's successfully completed our assets:precompile or not, let's just take the dumb approach for now and run it three times

Reviewed-on: https://code.launchpad.net/~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412557
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-is <email address hidden>

8ae33ea... by Barry Price on 2021-11-30

Clarify exactly which file our first run fails on

8bb9bff... by Barry Price on 2021-11-30

We can't tell from bundler's return code whether it's successfully completed our assets:precompile or not, let's just take the dumb approach for now and run it three times

c6cb23e... by Tom Haddon on 2021-11-29

Fix tox path conflicts

660f834... by Barry Price on 2021-11-29

Update LP:1903695 patch for v2.7.10 release

The code change remains the same, we just need to adjust the context around it due to this commit:

https://github.com/discourse/discourse/commit/2da0001965c6d8632d723c46ea5df9f22a1a23f1

Reviewed-on: https://code.launchpad.net/~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/412501
Reviewed-by: Benjamin Allot <email address hidden>