fix(#2758): set exact-o3r-version as default for all schematics if provided when adding @o3r/core

Author: matthieu-crouzet

packages/@o3r/application/schematics/ng-add/index.ts

@@ -26,14 +26,21 @@ function ngAddFn(options: NgAddSchematicsSchema): Rule {
       setupDependencies,
       getO3rPeerDeps,
       getProjectNewDependenciesTypes,
-      getPackageInstallConfig
+      getPackageInstallConfig,
+      getSchematicOptions
     } = await import('@o3r/schematics');
+    const workspaceConfig = getWorkspaceConfig(tree);
+    options = {
+      ...(workspaceConfig ? getSchematicOptions(workspaceConfig, context) : {}),
+      ...options
+    };
+
     const { isImported } = await import('@schematics/angular/utility/ast-utils');
     const ts = await import('typescript');
     const packageJsonPath = path.resolve(__dirname, '..', '..', 'package.json');
     const depsInfo = getO3rPeerDeps(packageJsonPath);
 
-    const workspaceProject = options.projectName ? getWorkspaceConfig(tree)?.projects[options.projectName] : undefined;
+    const workspaceProject = options.projectName ? workspaceConfig?.projects[options.projectName] : undefined;
 
     const addAngularAnimationPreferences: Rule = () => {
       const additionalRules: Rule[] = [];

packages/@o3r/application/schematics/ng-add/schema.json

@@ -13,8 +13,7 @@
     },
     "exactO3rVersion": {
       "type": "boolean",
-      "description": "Use a pinned version for otter packages",
-      "default": false
+      "description": "Use a pinned version for otter packages"
     }
   },
   "additionalProperties": true,

packages/@o3r/core/schematics/ng-add/index.ts

@@ -59,6 +59,8 @@ function ngAddFn(options: NgAddSchematicsSchema): Rule {
     return chain([
       // eslint-disable-next-line @typescript-eslint/naming-convention
       setupSchematicsParamsForProject({ '*:ng-add': { registerDevtool: options.withDevtool } }, options.projectName),
+      // eslint-disable-next-line @typescript-eslint/naming-convention
+      options.exactO3rVersion ? setupSchematicsParamsForProject({ '*:*': { exactO3rVersion: true } }, options.projectName) : noop(),
       options.projectName ? prepareProject(options, dependenciesSetupConfig) : noop(),
       registerPackageCollectionSchematics(corePackageJsonContent),
       async (t, c) => {

packages/@o3r/create/src/index.it.spec.ts

@@ -87,7 +87,7 @@ describe('Create new otter project command', () => {
     expect(() => packageManagerInstall(execInAppOptions)).not.toThrow();
 
     const appName = 'test-application';
-    expect(() => packageManagerExec({ script: 'ng', args: ['g', 'application', appName, '--exact-o3r-version'] }, execInAppOptions)).not.toThrow();
+    expect(() => packageManagerExec({ script: 'ng', args: ['g', 'application', appName] }, execInAppOptions)).not.toThrow();
     expect(existsSync(path.join(inProjectPath, 'project'))).toBe(false);
     expect(() => packageManagerRunOnProject(appName, true, { script: 'build' }, execInAppOptions)).not.toThrow();
 
@@ -98,8 +98,9 @@ describe('Create new otter project command', () => {
     [
       ...Object.entries(rootPackageJson.dependencies), ...Object.entries(rootPackageJson.devDependencies), ...Object.entries(resolutions),
       ...Object.entries(appPackageJson.dependencies), ...Object.entries(appPackageJson.devDependencies)
-    ].filter(([dep]) => dep.startsWith('@o3r/') || dep.startsWith('@ama-sdk/')).forEach(([,version]) => {
-      expect(version).toBe(o3rExactVersion);
+    ].filter(([dep]) => dep.startsWith('@o3r/') || dep.startsWith('@ama-sdk/')).forEach(([dep, version]) => {
+      // Adding `dep` to simplify the debug
+      expect(`${dep}=${version as string}`).toBe(`${dep}=${o3rExactVersion}`);
     });
   });
 });