Skip to content

fix: non prefix themes styling issue #4290

fix: non prefix themes styling issue

fix: non prefix themes styling issue #4290

Triggered via pull request August 29, 2024 15:26
Status Failure
Total duration 4m 12s
Artifacts 1

node.js.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 7 warnings
Tests
@tokens-studio/figma-plugin#test: command (/home/runner/work/figma-plugin/figma-plugin/packages/tokens-studio-for-figma) yarn run test exited (1)
Tests
Process completed with exit code 1.
Test coverage
Error: Command failed: cd packages/tokens-studio-for-figma && npm run test:coverage:ci && cd ../.. && mv packages/tokens-studio-for-figma/coverage-summary.json coverage-summary.json npm warn ignoring workspace config at /home/runner/work/figma-plugin/figma-plugin/packages/tokens-studio-for-figma/.npmrc npm warn ignoring workspace config at /home/runner/work/figma-plugin/figma-plugin/packages/tokens-studio-for-figma/.npmrc npm warn ignoring workspace config at /home/runner/work/figma-plugin/figma-plugin/packages/tokens-studio-for-figma/.npmrc ● Can set values on node › sets textstyle if matching Style is found and first part is ignored expect(jest.fn()).not.toHaveBeenCalled() Expected number of calls: 0 Received number of calls: 1 1: {"characters": "foobar", "fills": [], "fontName": {"family": "Inter", "style": "Regular"}, "getSharedPluginData": [Function getSharedPluginData], "id": "123:456", "setSharedPluginData": [Function setSharedPluginData], "textStyleId": undefined, "type": "TEXT"}, "type.heading.h1", "16" 334 | }, 335 | ); > 336 | expect(setTextValuesOnTargetSpy).not.toHaveBeenCalled(); | ^ 337 | expect(textNodeMock).toEqual({ ...textNodeMock, textStyleId: '456' }); 338 | }); 339 | at Object.<anonymous> (src/plugin/__tests__/setValuesOnNode.test.ts:336:42) ● Can set values on node › should be able to find a theme prefixed style expect(jest.fn()).not.toHaveBeenCalled() Expected number of calls: 0 Received number of calls: 1 1: {"bottomLeftRadius": 0, "bottomRightRadius": 0, "cornerRadius": 0, "dashPattern": [], "effectStyleId": "", "effects": [], "fillStyleId": "", "fills": [], "getSharedPluginData": [Function getSharedPluginData], "id": "123:457", "opacity": 1, "setSharedPluginData": [Function setSharedPluginData], "strokeStyleId": "", "strokeWeight": 0, "strokes": [], "topLeftRadius": 0, "topRightRadius": 0, "type": "RECTANGLE", "visible": true}, "shadows.default", "16" 454 | }, 455 | ); > 456 | expect(setEffectValuesOnTargetSpy).not.toHaveBeenCalled(); | ^ 457 | expect(solidNodeMock).toEqual({ ...solidNodeMock, effectStyleId: '123' }); 458 | }); 459 | at Object.<anonymous> (src/plugin/__tests__/setValuesOnNode.test.ts:456:44) ● getThemeReferences › should return the expected result expect(received).toEqual(expected) // deep equality - Expected - 0 + Received + 1 @@ -3,6 +3,7 @@ "token2" => "style1", }, "figmaVariableReferences": Map { "token1" => "variable1", }, + "potentialStylePathPrefixes": Array [], } 52 | const result = await getThemeReferences(); 53 | > 54 | expect(result).toEqual({ | ^ 55 | figmaStyleReferences: new Map([['token2', 'style1']]), 56 | figmaVariableReferences: new Map([['token1', 'variable1']]), 57 | }); at Object.<anonymous> (src/plugin/asyncMessageHandlers/__tests__/getThemeReferences.test.ts:54:20) ● getThemeReferences › should merge variable references with local references expect(received).toEqual(expected) // deep equality - Expected - 0 + Received + 1 @@ -5,6 +5,7 @@ }, "figmaVariableReferences": Map { "token1" => "variable1", "local.variable1" => "localVariable1", }, + "potentialStylePathPrefixes": Array [], } 76 | const result = await getThemeReferences(); 77 | > 78 | expect(result).toEqual({ | ^ 79 | figmaStyleReferences: new Map([ 80 | ['token2', 'style1'], 81 | ['local.style2', 'style2'], at Object.<anonymous> (src/plugin/asyncMessageHandlers/__tests__/getThemeReferences.test.ts:78:20) ● getThemeReferences › should respect theme variable references when conflict
Transformer Tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Performance
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Create Package
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Test coverage
The following actions uses node12 which is deprecated and will be forced to run on node16: anuraag016/Jest-Coverage-Diff@master. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test coverage
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3, anuraag016/Jest-Coverage-Diff@master. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Cypress
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Artifacts

Produced during runtime
Name Size
package
6.79 MB