refactor: adjust ApplicationContextMixin to use static class variable

Author: azech-hqs

src/context/index.js

@@ -1,6 +1,6 @@
 import { JSONSchemaFormDataProvider } from "./json_schema_provider";
 import {
-    ApplicationContextMixinBuilder,
+    ApplicationContextMixin,
     JobContextMixin,
     MaterialContextMixin,
     MaterialsContextMixin,
@@ -18,7 +18,7 @@ export {
     JobContextPickKeysForMixin,
     JSONSchemaFormDataProvider,
     WorkflowContextPickKeysForMixin,
-    ApplicationContextMixinBuilder,
+    ApplicationContextMixin,
     MaterialContextMixin,
     MaterialsContextMixin,
     MaterialsSetContextMixin,

src/context/mixins.js

@@ -3,14 +3,15 @@ import CryptoJS from "crypto-js";
 
 import { compareEntitiesInOrderedSetForSorting } from "../entity/set/ordered/utils";
 
-export const ApplicationContextMixinBuilder = (applicationCls) => (superclass) =>
+export const ApplicationContextMixin = (superclass) =>
     class extends superclass {
         constructor(config) {
             super(config);
-            if (!applicationCls)
-                throw Error("ApplicationContextMixinBuilder: applicationCls is undefined");
+            if (!this.constructor.applicationCls)
+                throw Error("ApplicationContextMixin: applicationCls is undefined");
             this._application =
-                (config.context && config.context.application) || applicationCls.createDefault();
+                (config.context && config.context.application) ||
+                this.constructor.applicationCls.createDefault();
         }
 
         get application() {

tests/context.tests.js

@@ -1,7 +1,7 @@
 import { expect } from "chai";
 import { mix } from "mixwith";
 
-import { MaterialContextMixin } from "../src/context/mixins";
+import { ApplicationContextMixin, MaterialContextMixin } from "../src/context/mixins";
 import { InMemoryEntity } from "../src/entity";
 
 class MockMaterial {
@@ -16,24 +16,45 @@ class SpecificMockMaterial {
     }
 }
 
-class ProviderEntity extends mix(InMemoryEntity).with(MaterialContextMixin) {
+class MockApplication {
+    static createDefault() {
+        return "defaultMockApplication";
+    }
+}
+
+class SpecificMockApplication {
+    static createDefault() {
+        return "defaultSpecificMockApplication";
+    }
+}
+
+class ProviderEntity extends mix(InMemoryEntity).with(
+    MaterialContextMixin,
+    ApplicationContextMixin,
+) {
     static materialCls = MockMaterial;
+
+    static applicationCls = MockApplication;
 }
 
 class DerivedProviderEntity extends ProviderEntity {
     static materialCls = SpecificMockMaterial;
+
+    static applicationCls = SpecificMockApplication;
 }
 
-describe("MaterialContextMixin", () => {
+describe("Material & Application ContextMixin", () => {
     const config = {};
 
-    it("uses static material class", () => {
+    it("uses static entity class", () => {
         const provider = new ProviderEntity(config);
         expect(provider.material).to.be.equal("defaultMockMaterial");
+        expect(provider.application).to.be.equal("defaultMockApplication");
     });
 
-    it("uses static material class from derived class", () => {
+    it("uses static entity class from derived class", () => {
         const provider = new DerivedProviderEntity(config);
         expect(provider.material).to.be.equal("defaultSpecificMockMaterial");
+        expect(provider.application).to.be.equal("defaultSpecificMockApplication");
     });
 });