fix: flaky unit test fix (#2142)

## Description
This PR fixes flaky test of ConfigurableActionsComponent 0060 



## Motivation
Karma/Jasmine runs tests in a random order, which causes failures in
tests that rely on a specific execution sequence.


## Fixes:
Please provide a list of the fixes implemented in this PR

* Items added


## Changes:
Please provide a list of the changes implemented by this PR

* changes made


## Tests included
- [ ] Included for each change/fix?
- [ ] Passing? (Merge will not be approved unless this is checked) 

## Documentation
- [ ] swagger documentation updated \[required\]
- [ ] official documentation updated \[nice-to-have\]

### official documentation info
If you have updated the official documentation, please provide PR # and
URL of the pages where the updates are included

## Backend version
- [ ] Does it require a specific version of the backend
- which version of the backend is required:

## Summary by Sourcery

Bug Fixes:
- Fix a flaky unit test by waiting for the component fixture to become
stable and refreshing the DOM before checking the number of actions.

Author: Junjiequan

karma.conf.js

@@ -23,6 +23,7 @@ module.exports = function (config) {
     },
     client: {
       jasmine: {
+        random: false,
         // you can add configuration options for Jasmine here
         // the possible options are listed at https://jasmine.github.io/api/edge/Configuration.html
         // for example, you can disable the random execution with `random: false`

src/app/shared/modules/configurable-actions/configurable-actions.component.spec.ts

@@ -80,17 +80,16 @@ describe("1010: ConfigurableActionsComponent", () => {
     }
   });
 
-  it("0020: actions should be visible when enabled in configuration", () => {
-    mockAppConfigService.appConfig.datafilesActionsEnabled = true;
-    expect(component.visible).toEqual(true);
-  });
-
-  it("0030: actions should be not visible when disabled in configuration", () => {
+  it("0020: actions should be not visible when disabled in configuration", () => {
     mockAppConfigService.appConfig.datafilesActionsEnabled = false;
-    //spyOn(mockAppConfigService, "getConfig").and.returnValue(localConfig);
     expect(component.visible).toEqual(false);
   });
 
+  it("0030: actions should be visible when enabled in configuration", () => {
+    mockAppConfigService.appConfig.datafilesActionsEnabled = true;
+    expect(component.visible).toEqual(true);
+  });
+
   it("0040: max file size should be the same as set in configuration, aka higher limit", () => {
     mockAppConfigService.appConfig.maxDirectDownloadSize =
       higherMaxFileSizeLimit;
@@ -110,6 +109,7 @@ describe("1010: ConfigurableActionsComponent", () => {
 
   it("0060: there should be as many actions as defined in default configuration", async () => {
     component.actionsConfig = mockActionsConfig;
+
     fixture.detectChanges();
 
     expect(component.sortedActionsConfig.length).toEqual(

src/app/shared/modules/configurable-actions/configurable-actions.component.ts

@@ -27,7 +27,7 @@ export class ConfigurableActionsComponent {
     if (!this.actionsConfig) {
       return [];
     }
-    return this.actionsConfig.sort((a: ActionConfig, b: ActionConfig) =>
+    return [...this.actionsConfig].sort((a: ActionConfig, b: ActionConfig) =>
       a.order && b.order ? a.order - b.order : 0,
     );
   }