fix: #33454 Site config was not set in the correct order #2

Merged

norb merged 6 commits from 33454/feat-site-settings-for-business-settings into master 2024-07-30 12:48:29 +00:00

Conversation 5 Commits 6 Files changed 5 +39 -7

andreas.steiner commented 2024-07-24 14:04:52 +00:00

Contributor

Copy link

@norb hier hatte es noch einen bug in den util-settings

@norb hier hatte es noch einen bug in den util-settings

andreas.steiner added 1 commit 2024-07-24 14:04:52 +00:00

fix: #33454 Site config was not set in the correct order 6f31664c05

andreas.steiner requested review from norb 2024-07-24 14:05:02 +00:00

norb was assigned by andreas.steiner 2024-07-24 14:05:13 +00:00

norb requested changes 2024-07-29 05:46:22 +00:00

norb left a comment

Owner

Copy link

hast du die unit tests laufen lassen? Die sollten das eigentlich testen, falls die entsprechenden Tests für die siteConf fehlen bitte ergänzen

hast du die unit tests laufen lassen? Die sollten das eigentlich testen, falls die entsprechenden Tests für die siteConf fehlen bitte ergänzen

andreas.steiner added 1 commit 2024-07-29 09:41:38 +00:00

fix: #33454 The localConf variable cannot be false by default; in which case the site test will fail 42921d0456

andreas.steiner added 1 commit 2024-07-29 10:53:11 +00:00

fix: #33454 Fix construct and load test 31735d8692

andreas.steiner commented 2024-07-29 10:53:36 +00:00

Author

Contributor

Copy link

hast du die unit tests laufen lassen? Die sollten das eigentlich testen, falls die entsprechenden Tests für die siteConf fehlen bitte ergänzen

Die tests laufen nun alle einwandfrei durch.

> hast du die unit tests laufen lassen? Die sollten das eigentlich testen, falls die entsprechenden Tests für die siteConf fehlen bitte ergänzen Die tests laufen nun alle einwandfrei durch.

norb commented 2024-07-29 14:14:51 +00:00

Owner

Copy link

wenn ich 31735d8692 in master cherry picke laufen die tests auch durch.
Bitte schreib einen test der die korrekte funktion von 6f31664c05 demonstriert. Der müsste ja dann folgerichtig auf dem master failen

wenn ich 31735d8692 in master cherry picke laufen die tests auch durch. Bitte schreib einen test der die korrekte funktion von 6f31664c05 demonstriert. Der müsste ja dann folgerichtig auf dem master failen

andreas.steiner added 1 commit 2024-07-29 16:32:20 +00:00

feat: #33454 Addition of a test to check the correct loading order d57c247860

andreas.steiner commented 2024-07-29 16:38:19 +00:00

Author

Contributor

Copy link

wenn ich 31735d8692 in master cherry picke laufen die tests auch durch.
Bitte schreib einen test der die korrekte funktion von 6f31664c05 demonstriert. Der müsste ja dann folgerichtig auf dem master failen

Der gewünschte Test kann wie folgt ausgeführt werden:
vendor/phpunit/phpunit/phpunit --filter testLoadingOrder tests/SettingsTest.php

Somit ist auch sichergestellt, dass die lade Reihenfolge korrekt ist.

> wenn ich 31735d8692 in master cherry picke laufen die tests auch durch. > Bitte schreib einen test der die korrekte funktion von 6f31664c05 demonstriert. Der müsste ja dann folgerichtig auf dem master failen Der gewünschte Test kann wie folgt ausgeführt werden: `vendor/phpunit/phpunit/phpunit --filter testLoadingOrder tests/SettingsTest.php` Somit ist auch sichergestellt, dass die lade Reihenfolge korrekt ist.

andreas.steiner added 1 commit 2024-07-29 16:45:01 +00:00

fix: #33454 Add missing files for the correct loading order test a58fe8454f

norb added 1 commit 2024-07-30 12:39:03 +00:00

fix: #33454 add config in default to be overwritten b1ca44843b

norb commented 2024-07-30 12:48:03 +00:00

Owner

Copy link

Im master läuft der Test jetzt wie erwartet nicht durch, da default nicht durch conf aus der site konfig überschrieben wurde:

There was 1 failure:

1) rabe\Util\tests\SettingsTest::testLoadingOrder
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'conf'
+'default'

/home/zeuz/repos/verua/Util-Settings/tests/SettingsTest.php:141

Im master läuft der Test jetzt wie erwartet nicht durch, da `default` nicht durch `conf` aus der site konfig überschrieben wurde: ``` There was 1 failure: 1) rabe\Util\tests\SettingsTest::testLoadingOrder Failed asserting that two strings are equal. --- Expected +++ Actual @@ @@ -'conf' +'default' /home/zeuz/repos/verua/Util-Settings/tests/SettingsTest.php:141 ```

norb merged commit e72fa80d63 into master 2024-07-30 12:48:29 +00:00

norb referenced this pull request from a commit 2024-07-30 12:48:31 +00:00

Merge pull request 'fix: #33454 Site config was not set in the correct order' (#2) from 33454/feat-site-settings-for-business-settings into master

Sign in to join this conversation.

Reviewers

No reviewers

norb

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

norb

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

rabe/Util-Settings!2

No description provided.