Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Generate mocks for data storage for use in tests #79

Merged
merged 3 commits into from
Apr 16, 2021
Merged

Conversation

katrogan
Copy link
Contributor

Signed-off-by: Katrina Rogan [email protected]

TL;DR

Generate mock for data storage for use in tests. Required to satisfy flyteorg/flytepropeller#254 (comment)

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

https://github.com/lyft/flyte/issues/

Follow-up issue

NA

Signed-off-by: Katrina Rogan <[email protected]>
@katrogan katrogan changed the title Generate mock for data storage for use in tests Generate mocks for data storage for use in tests Apr 16, 2021
@codecov
Copy link

codecov bot commented Apr 16, 2021

Codecov Report

Merging #79 (795d6d5) into master (15cac7b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   72.10%   72.10%           
=======================================
  Files          57       57           
  Lines        1975     1975           
=======================================
  Hits         1424     1424           
  Misses        412      412           
  Partials      139      139           
Flag Coverage Δ
unittests 72.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
storage/storage.go 60.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 15cac7b...795d6d5. Read the comment docs.

Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
@katrogan katrogan merged commit f0e2256 into master Apr 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants