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.

Sign up for GitHub

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

Introduced SMTP notification #5535

Merged
merged 33 commits into from
Sep 16, 2024

Using mocker 1.0.1

50ce320
Select commit
Loading
Failed to load commit list.
Merged

Introduced SMTP notification #5535

Using mocker 1.0.1
50ce320
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Sep 16, 2024 in 1s

48.36% of diff hit (target 36.21%)

View this Pull Request on Codecov

48.36% of diff hit (target 36.21%)

Annotations

Check warning on line 76 in flyteadmin/pkg/async/notifications/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/factory.go#L76 

Added line #L76 was not covered by tests

Check warning on line 78 in flyteadmin/pkg/async/notifications/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/factory.go#L78 

Added line #L78 was not covered by tests

Check warning on line 151 in flyteadmin/pkg/async/notifications/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/factory.go#L151 

Added line #L151 was not covered by tests

Check warning on line 196 in flyteadmin/pkg/async/notifications/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/factory.go#L196 

Added line #L196 was not covered by tests

Check warning on line 228 in flyteadmin/pkg/async/notifications/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/factory.go#L228 

Added line #L228 was not covered by tests

Check warning on line 63 in flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go#L63 

Added line #L63 was not covered by tests

Check warning on line 138 in flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go#L137-L138 

Added lines #L137 - L138 were not covered by tests

Check warning on line 156 in flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/implementations/smtp_emailer.go#L155-L156 

Added lines #L155 - L156 were not covered by tests

Check warning on line 24 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L23-L24 

Added lines #L23 - L24 were not covered by tests

Check warning on line 29 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L27-L29 

Added lines #L27 - L29 were not covered by tests

Check warning on line 34 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L32-L34 

Added lines #L32 - L34 were not covered by tests

Check warning on line 43 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L43 

Added line #L43 was not covered by tests

Check warning on line 56 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L55-L56 

Added lines #L55 - L56 were not covered by tests

Check warning on line 61 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L59-L61 

Added lines #L59 - L61 were not covered by tests

Check warning on line 66 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L64-L66 

Added lines #L64 - L66 were not covered by tests

Check warning on line 75 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L75 

Added line #L75 was not covered by tests

Check warning on line 88 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L87-L88 

Added lines #L87 - L88 were not covered by tests

Check warning on line 93 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L91-L93 

Added lines #L91 - L93 were not covered by tests

Check warning on line 98 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L96-L98 

Added lines #L96 - L98 were not covered by tests

Check warning on line 107 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L107 

Added line #L107 was not covered by tests

Check warning on line 116 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L116 

Added line #L116 was not covered by tests

Check warning on line 129 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L128-L129 

Added lines #L128 - L129 were not covered by tests

Check warning on line 134 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L132-L134 

Added lines #L132 - L134 were not covered by tests

Check warning on line 139 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L137-L139 

Added lines #L137 - L139 were not covered by tests

Check warning on line 148 in flyteadmin/pkg/async/notifications/mocks/smtp_client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/async/notifications/mocks/smtp_client.go#L148 

Added line #L148 was not covered by tests
View more details on Codecov