Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(104)

Side by Side Diff: components/offline_pages/background/save_page_request_unittest.cc

Issue 2053163002: [Offline pages] Adding persistent request queue based on SQLite (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebasing Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/offline_pages/background/save_page_request.h" 5 #include "components/offline_pages/background/save_page_request.h"
6 6
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace offline_pages { 9 namespace offline_pages {
10 10
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 ASSERT_EQ(kClientId, request.client_id()); 74 ASSERT_EQ(kClientId, request.client_id());
75 ASSERT_EQ(creation_time, request.creation_time()); 75 ASSERT_EQ(creation_time, request.creation_time());
76 ASSERT_EQ(activation_time, request.activation_time()); 76 ASSERT_EQ(activation_time, request.activation_time());
77 77
78 // Attempt time, attempt count and status will though. 78 // Attempt time, attempt count and status will though.
79 ASSERT_EQ(start_time, request.last_attempt_time()); 79 ASSERT_EQ(start_time, request.last_attempt_time());
80 ASSERT_EQ(1, request.attempt_count()); 80 ASSERT_EQ(1, request.attempt_count());
81 81
82 ASSERT_EQ(SavePageRequest::Status::STARTED, request.GetStatus(start_time)); 82 ASSERT_EQ(SavePageRequest::Status::STARTED, request.GetStatus(start_time));
83 83
84 request.MarkAttemptCompleted(); 84 base::Time completion_time = base::Time::Now();
85 request.MarkAttemptCompleted(completion_time);
85 86
86 // Again, most things don't change about the request. 87 // Again, most things don't change about the request.
87 ASSERT_EQ(kRequestId, request.request_id()); 88 ASSERT_EQ(kRequestId, request.request_id());
88 ASSERT_EQ(kUrl, request.url()); 89 ASSERT_EQ(kUrl, request.url());
89 ASSERT_EQ(kClientId, request.client_id()); 90 ASSERT_EQ(kClientId, request.client_id());
90 ASSERT_EQ(creation_time, request.creation_time()); 91 ASSERT_EQ(creation_time, request.creation_time());
91 ASSERT_EQ(activation_time, request.activation_time()); 92 ASSERT_EQ(activation_time, request.activation_time());
93 ASSERT_EQ(completion_time, request.last_attempt_time());
92 94
93 // Last attempt time and status will. Attempt count also unchanged. 95 // Last attempt time and status will. Attempt count also unchanged.
94 ASSERT_EQ(base::Time(), request.last_attempt_time()); 96 ASSERT_EQ(base::Time(), request.last_attempt_time());
95 ASSERT_EQ(1, request.attempt_count()); 97 ASSERT_EQ(1, request.attempt_count());
96 98
97 ASSERT_EQ(SavePageRequest::Status::PENDING, request.GetStatus(start_time)); 99 ASSERT_EQ(SavePageRequest::Status::PENDING, request.GetStatus(start_time));
98 } 100 }
99 101
100 } // offline_pages 102 } // offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698