Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add e2e tests for add-ons flow #3268

Closed
merelcht opened this issue Nov 3, 2023 · 1 comment · Fixed by #3325
Closed

Add e2e tests for add-ons flow #3268

merelcht opened this issue Nov 3, 2023 · 1 comment · Fixed by #3325
Assignees
Labels
Component: Testing Issue/PR that addresses tests

Comments

@merelcht
Copy link
Member

merelcht commented Nov 3, 2023

Description

For most of the add-ons flow I've been testing the work manually: create a new project with a combination of add-ons, install requirements and try a kedro run. This process has uncovered several bugs already.

Context

We should add automated tests to test at least some of the add-ons flow.

Implementation

At a minimum have tests for:

  1. kedro new --addons=none
  2. kedro new --addons=lint,docs,data,logging,... (everything but not viz and pyspark)
  3. kedro new --addons=all
  4. kedro new --addons=pyspark
  5. kedro new --addons=viz

All of the above should have an example so kedro run can be tested.

@merelcht merelcht added the Component: Testing Issue/PR that addresses tests label Nov 3, 2023
@merelcht merelcht removed this from Kedro-Viz Nov 3, 2023
@merelcht merelcht moved this to To Do in Kedro Framework Nov 7, 2023
@SajidAlamQB SajidAlamQB moved this from To Do to In Progress in Kedro Framework Nov 13, 2023
@SajidAlamQB SajidAlamQB moved this from In Progress to In Review in Kedro Framework Nov 27, 2023
@SajidAlamQB SajidAlamQB linked a pull request Nov 29, 2023 that will close this issue
7 tasks
@SajidAlamQB SajidAlamQB moved this from In Review to Done in Kedro Framework Nov 30, 2023
@SajidAlamQB
Copy link
Contributor

Completed in: #3325

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Testing Issue/PR that addresses tests
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants