Early return on access-sequence resolution fails

tools/projmgr/src/ProjMgrWorker.cpp

@@ -3395,8 +3395,12 @@ bool ProjMgrWorker::AddFile(const FileNode& src, vector<FileNode>& dst, ContextI
     FileNode srcNode = src;
 
     // Replace sequences and/or adjust file relative paths
-    ProcessSequenceRelative(context, srcNode.file, root);
-    ProcessSequencesRelatives(context, srcNode.build, root);
+    if (!ProcessSequenceRelative(context, srcNode.file, root)) {
+      return false;
+    }
+    if (!ProcessSequencesRelatives(context, srcNode.build, root)) {
+      return false;
+    }
     UpdateMisc(srcNode.build.misc, context.toolchain.name);
 
     // Set file category

tools/projmgr/test/data/TestAccessSequences/test-not_exisitng-access-sequences1.cproject.yml

@@ -0,0 +1,9 @@
+### CMSIS Project Description ###
+
+project:
+  components:
+    - component: CORE
+  groups:
+    - group: CMSE
+      files: 
+        - file: $cmse-lib(test-access-sequences-invalid)$

tools/projmgr/test/data/TestAccessSequences/test-not_exisitng-access-sequences1.csolution.yml

@@ -0,0 +1,25 @@
+### CMSIS Project Description ###
+solution:
+
+  target-types:
+    - type: CM0
+      device: RteTest_ARMCM0
+      misc:
+        - for-compiler: AC6
+          C-CPP: [-C-CPP-$Dname$]
+      define: 
+        - DEF-CM0-$Dname$-$Bname$
+      add-path:
+        - ./path/CM0/$Dname$
+
+  build-types:
+    - type: Debug
+      compiler: AC6
+      misc:
+        - C-CPP:
+            - -O1
+            - -g
+
+  projects:
+    - project: ./test-not_exisitng-access-sequences1.cproject.yml
+

tools/projmgr/test/data/TestAccessSequences/test-not_exisitng-access-sequences2.cproject.yml

@@ -0,0 +1,11 @@
+### CMSIS Project Description ###
+
+project:
+  components:
+    - component: CORE
+  groups:
+    - group: CMSE
+      files: 
+        - file: main.c
+          define:
+            - DEF2-PROJ1-$elf(test-access-sequences5+CM3)$

tools/projmgr/test/data/TestAccessSequences/test-not_exisitng-access-sequences2.csolution.yml

@@ -0,0 +1,25 @@
+### CMSIS Project Description ###
+solution:
+
+  target-types:
+    - type: CM0
+      device: RteTest_ARMCM0
+      misc:
+        - for-compiler: AC6
+          C-CPP: [-C-CPP-$Dname$]
+      define: 
+        - DEF-CM0-$Dname$-$Bname$
+      add-path:
+        - ./path/CM0/$Dname$
+
+  build-types:
+    - type: Debug
+      compiler: AC6
+      misc:
+        - C-CPP:
+            - -O1
+            - -g
+
+  projects:
+    - project: ./test-not_exisitng-access-sequences2.cproject.yml
+

tools/projmgr/test/src/ProjMgrUnitTests.cpp

@@ -2347,6 +2347,38 @@ TEST_F(ProjMgrUnitTests, AccessSequences2) {
    testinput_folder + "/TestAccessSequences/ref/test-access-sequences2.cbuild-idx.yml");
 }
 
+TEST_F(ProjMgrUnitTests, InvalidRefAccessSequences1) {
+  char* argv[7];
+  StdStreamRedirect streamRedirect;
+  // convert --solution solution.yml
+  const string& csolution = testinput_folder + "/TestAccessSequences/test-not_exisitng-access-sequences1.csolution.yml";
+  argv[1] = (char*)"convert";
+  argv[2] = (char*)"--solution";
+  argv[3] = (char*)csolution.c_str();
+  argv[4] = (char*)"-o";
+  argv[5] = (char*)testoutput_folder.c_str();
+  argv[6] = (char*)"--cbuildgen";
+  EXPECT_EQ(1, RunProjMgr(7, argv, m_envp));
+  const string& errStr = streamRedirect.GetErrorString();
+  EXPECT_TRUE(errStr.find("context 'test-access-sequences-invalid' referenced by access sequence 'cmse-lib' does not exist or is not selected") != string::npos);
+}
+
+TEST_F(ProjMgrUnitTests, InvalidRefAccessSequences2) {
+  char* argv[7];
+  StdStreamRedirect streamRedirect;
+  // convert --solution solution.yml
+  const string& csolution = testinput_folder + "/TestAccessSequences/test-not_exisitng-access-sequences2.csolution.yml";
+  argv[1] = (char*)"convert";
+  argv[2] = (char*)"--solution";
+  argv[3] = (char*)csolution.c_str();
+  argv[4] = (char*)"-o";
+  argv[5] = (char*)testoutput_folder.c_str();
+  argv[6] = (char*)"--cbuildgen";
+  EXPECT_EQ(1, RunProjMgr(7, argv, m_envp));
+  const string& errStr = streamRedirect.GetErrorString();
+  EXPECT_TRUE(errStr.find("context 'test-access-sequences5+CM3' referenced by access sequence 'elf' does not exist or is not selected") != string::npos);
+}
+
 TEST_F(ProjMgrUnitTests, PackAccessSequences) {
   char* argv[6];
   StdStreamRedirect streamRedirect;