Skip to content

Commit

Permalink
Fix report name + output widget
Browse files Browse the repository at this point in the history
  • Loading branch information
martinRenou committed Oct 16, 2024
1 parent b267e79 commit 091b3b7
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 10 deletions.
18 changes: 9 additions & 9 deletions .github/workflows/ui-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,38 +76,38 @@ jobs:
- name: Set test report name for progressive rendering
if: always() && matrix.progressive_rendering == true
run: |
echo "TEST_REPORT_NAME=progressive-voila-test-report-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_ASSETS_NAME=progressive-voila-test-assets-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_BENCHMARK_NAME=progressive-voila-test-benchmark-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_REPORT_NAME=progressive-voila-test-report-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
echo "TEST_ASSETS_NAME=progressive-voila-test-assets-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
echo "TEST_BENCHMARK_NAME=progressive-voila-test-benchmark-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
- name: Set test report name for non-progressive rendering
if: always() && matrix.progressive_rendering == false
run: |
echo "TEST_REPORT_NAME=voila-test-report-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_ASSETS_NAME=voila-test-assets-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_BENCHMARK_NAME=voila-test-benchmark-${{ matrix.jupyter-server-version }}" >> $GITHUB_ENV
echo "TEST_REPORT_NAME=voila-test-report-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
echo "TEST_ASSETS_NAME=voila-test-assets-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
echo "TEST_BENCHMARK_NAME=voila-test-benchmark-${{ matrix.jupyter-server-version }}-${{ matrix.ipywidgets }}" >> $GITHUB_ENV
- name: Upload Playwright Test assets
if: always()
uses: actions/upload-artifact@v4
with:
name: ${{ env.TEST_ASSETS_NAME }}${{ matrix.ipywidgets }}
name: ${{ env.TEST_ASSETS_NAME }}
path: |
ui-tests/test-results
- name: Upload Playwright Benchmark report
if: always()
uses: actions/upload-artifact@v4
with:
name: ${{ env.TEST_BENCHMARK_NAME }}${{ matrix.ipywidgets }}
name: ${{ env.TEST_BENCHMARK_NAME }}
path: |
ui-tests/benchmark-results
- name: Upload Playwright Test report
if: always()
uses: actions/upload-artifact@v4
with:
name: ${{ env.TEST_REPORT_NAME }}${{ matrix.ipywidgets }}
name: ${{ env.TEST_REPORT_NAME }}
path: |
ui-tests/playwright-report
Expand Down
8 changes: 7 additions & 1 deletion packages/widgets_manager7/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ import {
StyleView
} from '@jupyter-widgets/base';
import { JUPYTER_CONTROLS_VERSION } from '@jupyter-widgets/controls/lib/version';
import { WidgetRenderer } from '@jupyter-widgets/jupyterlab-manager';
import { WidgetRenderer, output } from '@jupyter-widgets/jupyterlab-manager';
import {
JupyterFrontEnd,
JupyterFrontEndPlugin
Expand Down Expand Up @@ -168,6 +168,12 @@ const widgetManager: JupyterFrontEndPlugin<IJupyterWidgetRegistry> = {
}
});

manager.register({
name: '@jupyter-widgets/output',
version: output.OUTPUT_WIDGET_VERSION,
exports: output as any
});

app.started.then(async () => {
await manager._loadFromKernel();
});
Expand Down

0 comments on commit 091b3b7

Please sign in to comment.