Update packages (#32)

CHANGELOG.md

@@ -1,3 +1,6 @@
+## 2.0.1
+* Update packages
+
 ## 2.0.0
 * Webpack migration
 

azure-pipelines.yml

@@ -23,20 +23,11 @@ steps:
     testResultsFormat: 'JUnit'
     testResultsFiles: '**/TESTS-*.xml'
     testRunTitle: 'Custom Visual unit-tests'
-    mergeTestResults: true # Optional
-    #searchFolder: '$(System.DefaultWorkingDirectory)' # Optional
-    #buildPlatform: # Optional
-    #buildConfiguration: # Optional
-    #publishRunAttachments: true # Optional
-
-- task: PublishCodeCoverageResults@1
-  inputs:
-    codeCoverageTool: 'cobertura' # Options: cobertura, jaCoCo
-    summaryFileLocation: '**/*-coverage.xml'
-    reportDirectory: '**/html-report'
     mergeTestResults: true
-    failIfCoverageEmpty: true
-    #summaryFileLocation: '$(System.DefaultWorkingDirectory)/**/*-coverage.xml'
-    #reportDirectory: '$(System.DefaultWorkingDirectory)/**/html-report'
-    #additionalCodeCoverageFiles: # Optional
-    #failIfCoverageEmpty: false # Optional
+
+#- task: PublishCodeCoverageResults@1
+#  inputs:
+#    codeCoverageTool: 'cobertura' # Options: cobertura, jaCoCo
+#    summaryFileLocation: '**/*-coverage.xml'
+#    reportDirectory: '**/html-report'
+#    failIfCoverageEmpty: true

karma.conf.ts

@@ -32,10 +32,7 @@ const path = require('path');
 
 const testRecursivePath = "test/visualTest.ts";
 const srcOriginalRecursivePath = "src/**/*.ts";
-const srcRecursivePath = ".tmp/drop/**/*.js";
 const coverageFolder = "coverage";
-const globals = "./test/globals.ts";
-
 
 process.env.CHROME_BIN = require("puppeteer").executablePath();
 
@@ -74,14 +71,13 @@ module.exports = (config: Config) => {
             reports: ["html", "lcovonly", "text-summary", "cobertura"],
             dir: path.join(__dirname, coverageFolder),
             'report-config': {
-                // all options available at: https://github.com/istanbuljs/istanbuljs/blob/aae256fb8b9a3d19414dcf069c592e88712c32c6/packages/istanbul-reports/lib/html/index.js#L135-L137
                 html: {
                     subdir: 'html-report'
                 }
             },
             combineBrowserReports: true,
             fixWebpackSourcePaths: true,
-            verbose: true // output config used by istanbul for debugging
+            verbose: false
         },
         singleRun: true,
         plugins: [
@@ -97,8 +93,6 @@ module.exports = (config: Config) => {
         files: [
             "node_modules/jquery/dist/jquery.min.js",
             "node_modules/jasmine-jquery/lib/jasmine-jquery.js",
-            globals,
-            srcRecursivePath,
             testRecursivePath,
             {
                 pattern: srcOriginalRecursivePath,
@@ -113,8 +107,7 @@ module.exports = (config: Config) => {
             }
         ],
         preprocessors: {
-            [testRecursivePath]: ["webpack"],
-            [srcRecursivePath]: ["webpack", "coverage"]
+            [testRecursivePath]: ["webpack", "coverage"]
         },
         typescriptPreprocessor: {
             options: tsconfig.compilerOptions

package.json

@@ -1,6 +1,6 @@
 {
   "name": "powerbi-visuals-chordchart",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "author": {
     "name": "Microsoft",
     "email": "pbicvsupport@microsoft.com"
@@ -20,45 +20,45 @@
     "url": "git+https://github.com/Microsoft/powerbi-visuals-chord.git"
   },
   "devDependencies": {
-    "@types/d3": "5.0.0",
-    "@types/jasmine-jquery": "1.5.28",
-    "@types/jasmine": "2.5.37",
-    "@types/jquery": "2.0.46",
-    "@types/lodash": "4.14.109",
-    "@types/webpack": "4.4.4",
-    "coveralls": "2.11.15",
+    "@types/d3": "5.0.1",
+    "@types/jasmine": "2.8.9",
+    "@types/jasmine-jquery": "1.5.32",
+    "@types/jquery": "3.3.22",
+    "@types/lodash": "4.14.117",
+    "@types/webpack": "4.4.17",
+    "coveralls": "3.0.2",
     "css-loader": "1.0.0",
     "istanbul-instrumenter-loader": "^3.0.1",
+    "jasmine": "3.2.0",
     "jasmine-jquery": "2.1.1",
-    "jasmine": "2.5.2",
-    "jquery": "3.2.1",
+    "jquery": "3.3.1",
+    "karma": "3.1.0",
     "karma-chrome-launcher": "2.2.0",
-    "karma-coverage-istanbul-reporter": "^2.0.4",
     "karma-coverage": "^1.1.2",
+    "karma-coverage-istanbul-reporter": "^2.0.4",
     "karma-jasmine": "1.1.2",
     "karma-junit-reporter": "^1.2.0",
     "karma-sourcemap-loader": "0.3.7",
-    "karma-typescript-preprocessor": "0.3.1",
-    "karma-typescript": "3.0.12",
-    "karma-webpack": "3.0.0",
-    "karma": "2.0.4",
+    "karma-typescript": "3.0.13",
+    "karma-typescript-preprocessor": "0.4.0",
+    "karma-webpack": "3.0.5",
+    "less": "3.8.1",
     "less-loader": "4.1.0",
-    "less": "2.1.1",
     "powerbi-models": "^1.0.3",
     "powerbi-visuals-api": "^2.1.0",
-    "powerbi-visuals-tools": "^3.0.1",
+    "powerbi-visuals-tools": "^3.0.4",
     "powerbi-visuals-utils-testutils": "^2.0.4",
     "puppeteer": "^1.5.0",
-    "style-loader": "0.21.0",
-    "ts-loader": "4.4.2",
-    "ts-node": "7.0.0",
-    "tslint": "4.0.2",
-    "typescript": "3.0.1",
-    "webpack": "4.14.0"
+    "style-loader": "0.23.1",
+    "ts-loader": "5.2.2",
+    "ts-node": "7.0.1",
+    "tslint": "5.11.0",
+    "typescript": "3.1.3",
+    "webpack": "4.22.0"
   },
   "dependencies": {
     "d3": "^5.7.0",
-    "lodash": "4.16.2",
+    "lodash": "4.17.11",
     "powerbi-visuals-utils-chartutils": "^2.0.7",
     "powerbi-visuals-utils-colorutils": "^2.0.2",
     "powerbi-visuals-utils-dataviewutils": "^2.0.1",

pbiviz.json

@@ -1,10 +1,10 @@
 {
   "visual": {
     "name": "ChordChart",
-    "displayName": "Chord 2.0.0",
+    "displayName": "Chord 2.0.1",
     "guid": "ChordChart1444757060245",
     "visualClassName": "ChordChart",
-    "version": "2.0.0",
+    "version": "2.0.1",
     "description": "A chord diagram is a graphical method of displaying the inter-relationships between data in a matrix.",
     "supportUrl": "http://community.powerbi.com",
     "gitHubUrl": "https://github.com/Microsoft/powerbi-visuals-chord"

src/chordChart.ts

@@ -214,7 +214,6 @@ export class ChordChart implements IVisual {
     public static defaultValue1: number = 1;
 
     /**
-     * 
      * @param dataView DataView object
      * @param host PBI Host object
      * @param colors Color Palette from PBI

test/visualTest.ts

@@ -335,8 +335,8 @@ describe("ChordChart", () => {
 
                 visualBuilder.updateFlushAllD3Transitions(dataView);
 
-                const slices: JQuery[] = visualBuilder.slices.toArray().map($),
-                    chords: JQuery[] = visualBuilder.chords.toArray().map($);
+                const slices: JQuery<any>[] = visualBuilder.slices.toArray().map($),
+                    chords: JQuery<any>[] = visualBuilder.chords.toArray().map($);
 
                 colors.forEach((color: string) => {
                     expect(doElementsUseColor(slices, color)).toBeTruthy();
@@ -486,8 +486,8 @@ describe("ChordChart", () => {
 
             it("should not use fill style", (done) => {
                 visualBuilder.updateRenderTimeout(dataView, () => {
-                    const slices: JQuery[] = visualBuilder.slices.toArray().map($);
-                    const chords: JQuery[] = visualBuilder.chords.toArray().map($);
+                    const slices: JQuery<any>[] = visualBuilder.slices.toArray().map($);
+                    const chords: JQuery<any>[] = visualBuilder.chords.toArray().map($);
 
                     expect(isColorAppliedToElements(slices, null, "fill"));
                     expect(isColorAppliedToElements(chords, null, "fill"));
@@ -498,8 +498,8 @@ describe("ChordChart", () => {
 
             it("should use stroke style", (done) => {
                 visualBuilder.updateRenderTimeout(dataView, () => {
-                    const slices: JQuery[] = visualBuilder.slices.toArray().map($);
-                    const chords: JQuery[] = visualBuilder.chords.toArray().map($);
+                    const slices: JQuery<any>[] = visualBuilder.slices.toArray().map($);
+                    const chords: JQuery<any>[] = visualBuilder.chords.toArray().map($);
 
                     expect(isColorAppliedToElements(slices, foregroundColor, "stroke"));
                     expect(isColorAppliedToElements(chords, foregroundColor, "stroke"));