Skip to content
Snippets Groups Projects
Commit bafda91d authored by Thomas Hutterer's avatar Thomas Hutterer :four:
Browse files

Create todos for expiring and expired SSH keys

Changelog: added
parent 7a012959
No related branches found
No related tags found
1 merge request!171826Create todos for expiring and expired SSH keys
Pipeline #1529833909 passed
Pipeline: rspec:predictive

#1529850367

    Pipeline: GitLab

    #1529850350

      Pipeline: Ruby 3.2.5 as-if-foss

      #1529836483

        ...@@ -15,10 +15,10 @@ def execute ...@@ -15,10 +15,10 @@ def execute
        return unless allowed? return unless allowed?
        if expiring_soon if expiring_soon
        create_expiring_soon_todos if Feature.enabled?(:todos_for_ssh_key_expiry, user) create_expiring_soon_todos
        trigger_expiring_soon_notification trigger_expiring_soon_notification
        else else
        create_expired_todos if Feature.enabled?(:todos_for_ssh_key_expiry, user) create_expired_todos
        trigger_expired_notification trigger_expired_notification
        end end
        end end
        ......
        ---
        name: todos_for_ssh_key_expiry
        feature_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/490325
        introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/166168
        rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/497334
        milestone: '17.5'
        group: group::personal productivity
        type: gitlab_com_derisk
        default_enabled: false
        ...@@ -73,14 +73,6 @@ ...@@ -73,14 +73,6 @@
        it 'updates notified column' do it 'updates notified column' do
        expect { subject.execute }.to change { key.reload.expiry_notification_delivered_at } expect { subject.execute }.to change { key.reload.expiry_notification_delivered_at }
        end end
        context 'when derisk feature flag is disabled' do
        before do
        stub_feature_flags(todos_for_ssh_key_expiry: false)
        end
        it_behaves_like 'does not create todo'
        end
        end end
        context 'when user does NOT have permission to receive notification' do context 'when user does NOT have permission to receive notification' do
        ...@@ -109,14 +101,6 @@ ...@@ -109,14 +101,6 @@
        it 'updates notified column' do it 'updates notified column' do
        expect { subject.execute }.to change { key.reload.before_expiry_notification_delivered_at } expect { subject.execute }.to change { key.reload.before_expiry_notification_delivered_at }
        end end
        context 'when derisk feature flag is disabled' do
        before do
        stub_feature_flags(todos_for_ssh_key_expiry: false)
        end
        it_behaves_like 'does not create todo'
        end
        end end
        context 'when user does NOT have permission to receive notification' do context 'when user does NOT have permission to receive notification' do
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Finish editing this message first!
        Please register or to comment