Merge pull request #960 from scramjetorg/fix/all_sequences

Type change in order to achieve coherent structure for displaying

Author: alicja-gruzdz

packages/adapters/src/docker-instance-adapter.ts

@@ -171,7 +171,7 @@ IComponent {
 
     // eslint-disable-next-line complexity
     async run(config: InstanceConfig, instancesServerPort: number, instanceId: string): Promise<ExitCode> {
-        if (config.type !== "docker") {
+        if (!(config.type === "docker" && "container" in config)) {
             throw new Error("Docker instance adapter run with invalid runner config");
         }
 

packages/types/src/runner-config.ts

@@ -5,7 +5,7 @@ import { InstanceArgs } from "./instance";
 import { PortConfig } from "./sequence-package-json";
 import { RunnerContainerConfiguration } from "./sth-configuration";
 
-type CommonSequenceConfig = {
+export type CommonSequenceConfig = {
     type: string;
     engines: Record<string, string>;
     id: string;
@@ -45,6 +45,7 @@ export type KubernetesSequenceConfig = CommonSequenceConfig & {
     type: "kubernetes"
 }
 
-export type SequenceConfig = DockerSequenceConfig | ProcessSequenceConfig | KubernetesSequenceConfig
+// eslint-disable-next-line max-len
+export type SequenceConfig = DockerSequenceConfig | ProcessSequenceConfig | KubernetesSequenceConfig | CommonSequenceConfig;
 
 export type InstanceConfig = SequenceConfig & { instanceAdapterExitDelay: number, limits: InstanceLimits }