Corrige l'assignation spot jour et nuit

Fusionne la branche 'BL3_Sanctuary_Bay' dans 'main'
Voir la demande de fusion Thoscellen/Wallset!11

.gitignore

@@ -4,4 +4,7 @@
 
 # Ignore python stuff
 **/__pycache__
-**/.pytest_cache
+**/.pytest_cache
+
+# Ignore les fichiers système de MacOS
+**/.DS_Store

.gitlab-ci.yml

@@ -9,8 +9,8 @@ stages:
   - Release
 
 variables:
-  ENTRY_FOLDER: $CI_PROJECT_DIR/test # test directory for testing CI/CD, or assets directory for normal operations
+  ENTRY_FOLDER: assets
-  RELEASE_VERSION: v0.$CI_PIPELINE_IID
+  RELEASE_VERSION: v$CI_PIPELINE_IID
   PACKAGE_REGISTRY_URL: "${CI_API_V4_URL}/projects/${CI_PROJECT_ID}/packages/generic/${PACKAGE_NAME}/${RELEASE_VERSION}"
 
 # Jobs from here run on Merge Requests as prerequisite for merging.
@@ -20,14 +20,13 @@ variables:
   rules:
     - if: $CI_COMMIT_TAG
       when: never
-    - if: $CI_MERGE_REQUEST_SOURCE_BRANCH_NAME == "cicd"
     - changes:
         - ${ENTRY_FOLDER}/${PACKAGE_NAME}/*
   script:
     - cd src/
     - npm install
     - npm run start
-    - cat ${ENTRY_FOLDER}/${PACKAGE_NAME}/theme.json
+    - cat ../${ENTRY_FOLDER}/${PACKAGE_NAME}/theme.json
     - npm run test
   artifacts:
     paths:
@@ -39,7 +38,6 @@ variables:
   stage: Test
   image: python:latest
   rules:
-    - if: $CI_MERGE_REQUEST_SOURCE_BRANCH_NAME == "cicd"
     - if: $CI_PIPELINE_SOURCE == "merge_request_event"
       changes:
         - ${ENTRY_FOLDER}/${PACKAGE_NAME}/*
@@ -55,7 +53,6 @@ variables:
   rules:
     - if: $CI_COMMIT_TAG
       when: never
-    - if: $CI_MERGE_REQUEST_SOURCE_BRANCH_NAME == "cicd"
     - if: $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
       changes:
         - ${ENTRY_FOLDER}/${PACKAGE_NAME}/*

.gitlab/BL2_frozen_wastelands.yml

@@ -0,0 +1,32 @@
+manifest-job/BL2_frozen_wastelands:
+  variables:
+    PACKAGE_NAME: "BL2_frozen_wastelands"
+  extends: .manifest-job
+
+test-images-job/BL2_frozen_wastelands:
+  needs:
+    - manifest-job/BL2_frozen_wastelands
+  variables:
+    PACKAGE_NAME: "BL2_frozen_wastelands"
+  extends: .test-job
+
+bundle-job/BL2_frozen_wastelands:
+  needs:
+    - manifest-job/BL2_frozen_wastelands
+  variables:
+    PACKAGE_NAME: "BL2_frozen_wastelands"
+  extends: .bundle-job
+
+upload-job/BL2_frozen_wastelands:
+  needs:
+    - bundle-job/BL2_frozen_wastelands
+  variables:
+    PACKAGE_NAME: "BL2_frozen_wastelands"
+  extends: .upload-job
+
+release-job/BL2_frozen_wastelands:
+  needs:
+    - upload-job/BL2_frozen_wastelands
+  variables:
+    PACKAGE_NAME: "BL2_frozen_wastelands"
+  extends: .release-job

.gitlab/BL3_Sanctuary_Bay.yml

@@ -0,0 +1,32 @@
+manifest-job/BL3_Sanctuary_Bay:
+  variables:
+    PACKAGE_NAME: "BL3_Sanctuary_Bay"
+  extends: .manifest-job
+
+test-images-job/BL3_Sanctuary_Bay:
+  needs:
+    - manifest-job/BL3_Sanctuary_Bay
+  variables:
+    PACKAGE_NAME: "BL3_Sanctuary_Bay"
+  extends: .test-job
+
+bundle-job/BL3_Sanctuary_Bay:
+  needs:
+    - manifest-job/BL3_Sanctuary_Bay
+  variables:
+    PACKAGE_NAME: "BL3_Sanctuary_Bay"
+  extends: .bundle-job
+
+upload-job/BL3_Sanctuary_Bay:
+  needs:
+    - bundle-job/BL3_Sanctuary_Bay
+  variables:
+    PACKAGE_NAME: "BL3_Sanctuary_Bay"
+  extends: .upload-job
+
+release-job/BL3_Sanctuary_Bay:
+  needs:
+    - upload-job/BL3_Sanctuary_Bay
+  variables:
+    PACKAGE_NAME: "BL3_Sanctuary_Bay"
+  extends: .release-job

.gitlab/existing_valid_theme_set.yml

@@ -1,32 +0,0 @@
-manifest-job/existing_valid_theme_set:
-  variables:
-    PACKAGE_NAME: "existing_valid_theme_set"
-  extends: .manifest-job
-
-test-images-job/existing_valid_theme_set:
-  needs:
-    - manifest-job/existing_valid_theme_set
-  variables:
-    PACKAGE_NAME: "existing_valid_theme_set"
-  extends: .test-job
-
-bundle-job/existing_valid_theme_set:
-  needs:
-    - manifest-job/existing_valid_theme_set
-  variables:
-    PACKAGE_NAME: "existing_valid_theme_set"
-  extends: .bundle-job
-
-upload-job/existing_valid_theme_set:
-  needs:
-    - bundle-job/existing_valid_theme_set
-  variables:
-    PACKAGE_NAME: "existing_valid_theme_set"
-  extends: .upload-job
-
-release-job/existing_valid_theme_set:
-  needs:
-    - upload-job/existing_valid_theme_set
-  variables:
-    PACKAGE_NAME: "existing_valid_theme_set"
-  extends: .release-job

.gitlab/missing_tag_set.yml

@@ -1,32 +0,0 @@
-manifest-job/missing_tag_set:
-  variables:
-    PACKAGE_NAME: "missing_tag_set"
-  extends: .manifest-job
-
-test-images-job/missing_tag_set:
-  needs:
-    - manifest-job/missing_tag_set
-  variables:
-    PACKAGE_NAME: "missing_tag_set"
-  extends: .test-job
-
-bundle-job/missing_tag_set:
-  needs:
-    - manifest-job/missing_tag_set
-  variables:
-    PACKAGE_NAME: "missing_tag_set"
-  extends: .bundle-job
-
-upload-job/missing_tag_set:
-  needs:
-    - bundle-job/missing_tag_set
-  variables:
-    PACKAGE_NAME: "missing_tag_set"
-  extends: .upload-job
-
-release-job/missing_tag_set:
-  needs:
-    - upload-job/missing_tag_set
-  variables:
-    PACKAGE_NAME: "missing_tag_set"
-  extends: .release-job

.gitlab/unbalanced_set.yml

@@ -1,32 +0,0 @@
-manifest-job/unbalanced_set:
-  variables:
-    PACKAGE_NAME: "unbalanced_set"
-  extends: .manifest-job
-
-test-images-job/unbalanced_set:
-  needs:
-    - manifest-job/unbalanced_set
-  variables:
-    PACKAGE_NAME: "unbalanced_set"
-  extends: .test-job
-
-bundle-job/unbalanced_set:
-  needs:
-    - manifest-job/unbalanced_set
-  variables:
-    PACKAGE_NAME: "unbalanced_set"
-  extends: .bundle-job
-
-upload-job/unbalanced_set:
-  needs:
-    - bundle-job/unbalanced_set
-  variables:
-    PACKAGE_NAME: "unbalanced_set"
-  extends: .upload-job
-
-release-job/unbalanced_set:
-  needs:
-    - upload-job/unbalanced_set
-  variables:
-    PACKAGE_NAME: "unbalanced_set"
-  extends: .release-job

.gitlab/valid_set.yml

@@ -1,32 +0,0 @@
-manifest-job/valid_set:
-  variables:
-    PACKAGE_NAME: "valid_set"
-  extends: .manifest-job
-
-test-images-job/valid_set:
-  needs:
-    - manifest-job/valid_set
-  variables:
-    PACKAGE_NAME: "valid_set"
-  extends: .test-job
-
-bundle-job/valid_set:
-  needs:
-    - manifest-job/valid_set
-  variables:
-    PACKAGE_NAME: "valid_set"
-  extends: .bundle-job
-
-upload-job/valid_set:
-  needs:
-    - bundle-job/valid_set
-  variables:
-    PACKAGE_NAME: "valid_set"
-  extends: .upload-job
-
-release-job/valid_set:
-  needs:
-    - upload-job/valid_set
-  variables:
-    PACKAGE_NAME: "valid_set"
-  extends: .release-job

.vscode/launch.json

@@ -12,7 +12,8 @@
 				"start"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "valid_set"
 			},
 			"skipFiles": [
@@ -30,7 +31,8 @@
 				"test"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "valid_set"
 			},
 			"skipFiles": [
@@ -51,7 +53,8 @@
 				"--color=yes"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "valid_set"
 			},
 			"justMyCode": true
@@ -67,7 +70,8 @@
 				"start"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "existing_valid_theme_set"
 			},
 			"skipFiles": [
@@ -85,7 +89,8 @@
 				"test"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "existing_valid_theme_set"
 			},
 			"skipFiles": [
@@ -106,7 +111,8 @@
 				"--color=yes"
 			],
 			"env": {
-				"ENTRY_FOLDER": "${workspaceFolder}/test",
+				"CI_PROJECT_DIR": "${workspaceFolder}",
+				"ENTRY_FOLDER": "test",
 				"PACKAGE_NAME": "existing_valid_theme_set"
 			},
 			"justMyCode": true

src/build-manifest.js

@@ -5,10 +5,11 @@ const { exit } = require("process");
 
 
 // Fetch envs to target the right folder
-const slug = process.env["PACKAGE_NAME"];
+const workDir = process.env["CI_PROJECT_DIR"];
 const root = process.env["ENTRY_FOLDER"];
-if (!slug || !root) exit(1);
+const slug = process.env["PACKAGE_NAME"];
-let workingDirectory = path.join(root, slug);
+if (!workDir || !slug || !root) exit(1);
+let workingDirectory = path.join(workDir, root, slug);
 
 // extract metadata from them
 fs.readdir(workingDirectory, (err, files) => {

src/manifest.spec.js

@@ -33,11 +33,13 @@ function flatReferences(manifest) {
 
 describe('Mandatory Checks', function () {
     before(function () {
+        const workDir = process.env["CI_PROJECT_DIR"];
+        expect(workDir).to.be.a("string").that.is.not.empty;
         const slug = process.env["PACKAGE_NAME"];
         expect(slug).to.be.a("string").that.is.not.empty;
         const root = process.env["ENTRY_FOLDER"];
         expect(root).to.be.a("string").that.is.not.empty;
-        workingDirectory = path.join(root, slug);
+        workingDirectory = path.join(workDir, root, slug);
         files = fs.readdirSync(workingDirectory);
         expect(files).to.be.an('array').that.is.not.empty;
         validator = ajv.compile(JSON.parse(fs.readFileSync(themeSchemaFile, 'utf8')));

src/test_images.py

@@ -40,9 +40,10 @@ def _validate_brightness_image(working_path, theme_config, high_light, image_lis
 @pytest.fixture
 def working_path():
 	# get the global variables containing gitlab-given project slug.
+	work_directory = os.environ["CI_PROJECT_DIR"]
+	entry_folder = os.environ["ENTRY_FOLDER"]
 	project_slug = os.environ["PACKAGE_NAME"]
-	root_folder = os.environ["ENTRY_FOLDER"]
+	working_path = Path(work_directory, entry_folder, project_slug)
-	working_path = Path(root_folder, project_slug)
 	if not working_path.is_dir():
 		raise FileNotFoundError(f"No project found for the given {working_path}.")
 	return working_path

test/existing_valid_theme_set/theme.json

@@ -2,12 +2,12 @@
 	"dayImageList": [
 		1
 	],
+	"imageFilename": "stray__*.jpg",
 	"nightImageList": [
 		2
 	],
-	"sunsetImageList": [],
-	"sunriseImageList": [],
-	"imageFilename": "stray__*.jpg",
 	"imageCredits": "©Thoscellen, ©BlueTwelve",
-	"displayName": "Stray"
+	"sunriseImageList": [],
+	"displayName": "Stray",
+	"sunsetImageList": []
 }

test/missing_tag_set/theme.json

@@ -1,7 +1,7 @@
 {
     "imageFilename": "bl3_landscape_*.jpg",
+    "displayName": "BL3",
     "imageCredits": "© Gearbox",
-    "displayName": "",
     "dayImageList": [
         2
     ],