Remove plotly.js-dist, tests (WIP)

Author: bzh2610

package-lock.json

@@ -19,7 +19,6 @@
     "@angular/platform-browser": "^20.0.2",
     "@angular/platform-browser-dynamic": "^20.0.2",
     "@angular/router": "^20.0.2",
-    "plotly.js-dist": "^2.20.0",
     "plotly.js-dist-min": "^3.1.1",
     "rxjs": "~7.8.0",
     "tslib": "^2.8.1",

package.json

@@ -1,11 +1,10 @@
 import { NgModule } from '@angular/core';
 import { BrowserModule } from '@angular/platform-browser';
-import * as PlotlyJS from 'plotly.js-dist';
+import * as PlotlyJS from "plotly.js-dist-min";
+
 import { PlotlyModule } from '../../../plotly/src/lib/plotly.module';
 import { PlotlyViaCDNModule } from 'projects/plotly/src/lib/plotly-via-cdn.module';
 
-
-
 import { AppComponent } from './app.component';
 import { PlotlyViaWindowModule } from 'projects/plotly/src/lib/plotly-via-window.module';
 

projects/demo_app/src/app/app.module.ts

@@ -83,7 +83,7 @@ describe('PlotlyComponent', () => {
             component.ngDoCheck();
             expect(component.updatePlot).toHaveBeenCalledTimes(1);
 
-            component.data()[0].y[0] = 12;
+            component.data()[0]['y'][0] = 12;
             component.ngDoCheck();
             expect(component.updatePlot).toHaveBeenCalledTimes(2);
             done();
@@ -97,14 +97,14 @@ describe('PlotlyComponent', () => {
             component.ngDoCheck();
             expect(component.updatePlot).not.toHaveBeenCalled();
 
-            componentRef.setInput('layout', { title: 'title two' });
+            componentRef.setInput('layout', { title: { text: 'title two' }});
             component.ngDoCheck();
             expect(component.updatePlot).toHaveBeenCalled();
 
             component.ngDoCheck();
             expect(component.updatePlot).toHaveBeenCalledTimes(1);
 
-            component.layout()['title'] = 'title three ';
+            component.layout().title = { text: 'title three '};
             component.ngDoCheck();
             expect(component.updatePlot).toHaveBeenCalledTimes(2);
             done();
@@ -197,21 +197,21 @@ describe('PlotlyComponent', () => {
         await component.createPlot();
         await fixture.whenStable();
 
-        spyOn(Plots, 'resize').and.callThrough();
+        const spy = spyOn(Plots, 'resize').and.callThrough();
 
         window.dispatchEvent(new Event('resize'));
         await fixture.whenStable();
 
-        expect(Plots.resize).toHaveBeenCalled();
-        Plots.resize.calls.reset();
+        expect(spy).toHaveBeenCalled();
+        spy.calls.reset();
 
         fixture.destroy();
         await fixture.whenStable();
 
         window.dispatchEvent(new Event('resize'));
         await fixture.whenStable();
 
-        expect(Plots.resize).not.toHaveBeenCalled();
+        expect(spy).not.toHaveBeenCalled();
     });
 
 

projects/plotly/src/lib/plotly.component.spec.ts

@@ -1,7 +1,7 @@
 import { TestBed, inject } from '@angular/core/testing';
-import PlotlyJS from 'plotly.js-dist';
 
 import { PlotlyService } from './plotly.service';
+import * as PlotlyJS from "plotly.js-dist-min";
 import {PlotlyHTMLElement} from "plotly.js-dist-min";
 
 describe('PlotlyService', () => {