Skip to content

Conversation

karangattu
Copy link
Collaborator

WHAT?

Add E2E tests for accordion and the input text area autoresizing.

HOW?

  1. Add corresponding classes for components used in the tests.
  2. Add example apps used for each test in their own directory.
  3. Uncomment code in the tests that are blocked due to existing bugs.

@CLAassistant
Copy link

CLAassistant commented Jul 7, 2023

CLA assistant check
All committers have signed the CLA.

@schloerke
Copy link
Collaborator

@karangattu Given conversation in #605 , let's just use the pixel height and assert that they are not the same values. If we can parse them as int/float, then assert that the second value is bigger than the first.

@karangattu karangattu marked this pull request as ready for review July 13, 2023 05:17
@karangattu karangattu requested a review from schloerke July 13, 2023 05:17
@karangattu karangattu merged commit a50c637 into main Jul 21, 2023
@karangattu karangattu deleted the add-accordion-test branch July 21, 2023 20:29
schloerke added a commit that referenced this pull request Jul 24, 2023
* main:
  Add E2E tests for accordion and autoresize (#601)
  Add card test (#622)
  Add sidebar test (#631)
  Make card fullscreen icon a tooltip (#632)
  Pull in changes from rstudio/bslib#697 and rstudio/bslib#699
  Changelog tweak. Followup to #629
  Add experimental tooltip methods and example apps (#629)
  Use `blib::bs_theme(5,"shiny")` for py-shiny theme (#624)
  Rename shiny/examples to shiny/api-examples (and X/examples to X/api-examples) (#627)
  Make --app-path work with app file argument (#598)
  Don't eval example code block
  Fix example code blocks (#626)
  Annotation export example (#584)
  Add todo list example (#603)
schloerke added a commit that referenced this pull request Aug 10, 2023
* main: (26 commits)
  Use a renderer generator to create sync/async render methods with full typing (#621)
  docs: Add Tooltip section in experimental (#673)
  Bump version to 0.5.1
  Add missing sidebar stylesheet dep (#667)
  GitHub action cleanup (#662)
  Add a triage label to issues created by non-maintainers (#647)
  Bump version to 0.5.0.9000
  Model score example (#650)
  Fix malformed changelog entry
  Update CHANGELOG.md for data frame filters (#654)
  Bump version to 0.5.0
  Allow ui.update_slider to handle non-numeric values (#649)
  Add updated path to examples (#645)
  Bump griffe version
  Update README
  Fix jumpy cursor in numeric filter (#640)
  Build API docs (#633)
  Pyright: ignore docs dir
  Filtering feature for data grid/table (#592)
  Add E2E tests for accordion and autoresize (#601)
  ...
schloerke added a commit that referenced this pull request Aug 10, 2023
* main: (40 commits)
  Use a renderer generator to create sync/async render methods with full typing (#621)
  docs: Add Tooltip section in experimental (#673)
  Bump version to 0.5.1
  Add missing sidebar stylesheet dep (#667)
  GitHub action cleanup (#662)
  Add a triage label to issues created by non-maintainers (#647)
  Bump version to 0.5.0.9000
  Model score example (#650)
  Fix malformed changelog entry
  Update CHANGELOG.md for data frame filters (#654)
  Bump version to 0.5.0
  Allow ui.update_slider to handle non-numeric values (#649)
  Add updated path to examples (#645)
  Bump griffe version
  Update README
  Fix jumpy cursor in numeric filter (#640)
  Build API docs (#633)
  Pyright: ignore docs dir
  Filtering feature for data grid/table (#592)
  Add E2E tests for accordion and autoresize (#601)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants