12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546254725482549255025512552255325542555255625572558255925602561256225632564256525662567256825692570257125722573257425752576257725782579258025812582258325842585258625872588258925902591259225932594259525962597259825992600260126022603260426052606260726082609261026112612261326142615261626172618261926202621262226232624262526262627262826292630263126322633263426352636263726382639264026412642264326442645264626472648264926502651265226532654265526562657265826592660266126622663266426652666266726682669267026712672267326742675267626772678267926802681268226832684268526862687268826892690269126922693269426952696269726982699270027012702270327042705270627072708270927102711271227132714271527162717271827192720272127222723272427252726272727282729273027312732273327342735273627372738273927402741274227432744274527462747274827492750275127522753275427552756275727582759276027612762276327642765276627672768276927702771277227732774277527762777277827792780278127822783278427852786278727882789279027912792279327942795279627972798279928002801280228032804280528062807280828092810281128122813281428152816281728182819282028212822282328242825282628272828282928302831283228332834283528362837283828392840284128422843284428452846284728482849285028512852285328542855285628572858285928602861286228632864286528662867286828692870287128722873287428752876287728782879288028812882288328842885288628872888288928902891289228932894289528962897289828992900290129022903290429052906290729082909291029112912291329142915291629172918291929202921292229232924292529262927292829292930293129322933293429352936293729382939294029412942294329442945294629472948294929502951295229532954295529562957295829592960296129622963296429652966296729682969297029712972297329742975297629772978297929802981298229832984298529862987298829892990299129922993299429952996299729982999300030013002300330043005300630073008300930103011301230133014301530163017301830193020302130223023302430253026302730283029303030313032303330343035303630373038303930403041304230433044304530463047304830493050305130523053305430553056305730583059306030613062306330643065306630673068306930703071307230733074307530763077307830793080308130823083308430853086308730883089309030913092309330943095309630973098309931003101310231033104310531063107310831093110311131123113311431153116311731183119312031213122312331243125312631273128312931303131313231333134313531363137313831393140314131423143314431453146314731483149315031513152315331543155315631573158315931603161316231633164316531663167316831693170317131723173317431753176317731783179318031813182318331843185318631873188318931903191319231933194319531963197319831993200320132023203320432053206320732083209321032113212321332143215321632173218321932203221322232233224322532263227322832293230323132323233323432353236323732383239324032413242324332443245324632473248324932503251325232533254325532563257325832593260326132623263326432653266326732683269327032713272327332743275327632773278327932803281328232833284328532863287328832893290329132923293329432953296329732983299330033013302330333043305330633073308330933103311331233133314331533163317331833193320332133223323332433253326332733283329333033313332333333343335333633373338333933403341334233433344334533463347334833493350335133523353335433553356335733583359336033613362336333643365336633673368336933703371337233733374337533763377337833793380338133823383338433853386338733883389339033913392339333943395339633973398339934003401340234033404340534063407340834093410341134123413341434153416341734183419342034213422342334243425342634273428342934303431343234333434343534363437343834393440344134423443344434453446344734483449345034513452345334543455345634573458345934603461346234633464346534663467346834693470347134723473347434753476347734783479348034813482348334843485348634873488348934903491349234933494349534963497349834993500350135023503350435053506350735083509351035113512351335143515351635173518351935203521352235233524352535263527352835293530353135323533353435353536353735383539354035413542354335443545354635473548354935503551355235533554355535563557355835593560356135623563356435653566356735683569357035713572357335743575357635773578357935803581358235833584358535863587358835893590359135923593359435953596359735983599360036013602360336043605360636073608360936103611361236133614361536163617361836193620362136223623362436253626362736283629363036313632363336343635363636373638363936403641364236433644364536463647364836493650365136523653365436553656365736583659366036613662366336643665366636673668366936703671367236733674367536763677367836793680368136823683368436853686368736883689369036913692369336943695369636973698369937003701370237033704370537063707370837093710371137123713371437153716371737183719372037213722372337243725372637273728372937303731373237333734373537363737373837393740374137423743374437453746374737483749375037513752375337543755375637573758375937603761376237633764376537663767376837693770377137723773377437753776377737783779378037813782378337843785378637873788378937903791379237933794379537963797379837993800380138023803380438053806380738083809381038113812381338143815381638173818381938203821382238233824382538263827382838293830383138323833383438353836383738383839384038413842384338443845384638473848384938503851385238533854385538563857385838593860386138623863386438653866386738683869387038713872387338743875387638773878387938803881388238833884388538863887388838893890389138923893389438953896389738983899390039013902390339043905390639073908390939103911391239133914391539163917391839193920392139223923392439253926392739283929393039313932393339343935393639373938393939403941394239433944394539463947394839493950395139523953395439553956395739583959396039613962396339643965396639673968396939703971397239733974397539763977397839793980398139823983398439853986398739883989399039913992399339943995399639973998399940004001400240034004400540064007400840094010401140124013401440154016401740184019402040214022402340244025402640274028402940304031403240334034403540364037403840394040404140424043404440454046404740484049405040514052405340544055405640574058405940604061406240634064406540664067406840694070407140724073407440754076407740784079408040814082408340844085408640874088408940904091409240934094409540964097409840994100410141024103410441054106410741084109411041114112411341144115411641174118411941204121412241234124412541264127412841294130413141324133413441354136413741384139414041414142414341444145414641474148414941504151415241534154415541564157415841594160416141624163416441654166416741684169417041714172417341744175417641774178417941804181418241834184418541864187418841894190419141924193419441954196419741984199420042014202420342044205420642074208420942104211421242134214421542164217421842194220422142224223422442254226422742284229423042314232423342344235423642374238423942404241424242434244424542464247424842494250425142524253425442554256425742584259426042614262426342644265426642674268426942704271427242734274427542764277427842794280428142824283428442854286428742884289429042914292429342944295429642974298429943004301430243034304430543064307430843094310431143124313431443154316431743184319432043214322432343244325432643274328432943304331433243334334433543364337433843394340434143424343434443454346434743484349435043514352435343544355435643574358435943604361436243634364436543664367436843694370437143724373437443754376437743784379438043814382438343844385438643874388438943904391439243934394439543964397439843994400440144024403440444054406440744084409441044114412441344144415441644174418441944204421442244234424442544264427442844294430443144324433443444354436443744384439444044414442444344444445444644474448444944504451445244534454445544564457445844594460446144624463446444654466446744684469447044714472447344744475447644774478447944804481448244834484448544864487448844894490449144924493449444954496449744984499450045014502450345044505450645074508450945104511451245134514451545164517451845194520452145224523452445254526452745284529453045314532453345344535453645374538453945404541454245434544454545464547454845494550455145524553455445554556455745584559456045614562456345644565456645674568456945704571457245734574457545764577457845794580458145824583458445854586458745884589459045914592459345944595459645974598459946004601460246034604460546064607460846094610461146124613461446154616461746184619462046214622462346244625462646274628462946304631463246334634463546364637463846394640464146424643464446454646464746484649465046514652465346544655465646574658465946604661466246634664466546664667466846694670467146724673467446754676467746784679468046814682468346844685468646874688468946904691469246934694469546964697469846994700470147024703470447054706470747084709471047114712471347144715471647174718471947204721472247234724472547264727472847294730473147324733473447354736473747384739474047414742474347444745474647474748474947504751475247534754475547564757475847594760476147624763476447654766476747684769477047714772477347744775477647774778477947804781478247834784478547864787478847894790479147924793479447954796479747984799480048014802480348044805480648074808480948104811481248134814481548164817481848194820482148224823482448254826482748284829483048314832483348344835483648374838483948404841484248434844484548464847484848494850485148524853485448554856485748584859486048614862486348644865486648674868486948704871487248734874487548764877487848794880488148824883488448854886488748884889489048914892489348944895489648974898489949004901490249034904490549064907490849094910491149124913491449154916491749184919492049214922492349244925492649274928492949304931493249334934493549364937493849394940494149424943494449454946494749484949495049514952495349544955495649574958495949604961496249634964496549664967496849694970497149724973497449754976497749784979498049814982498349844985498649874988498949904991499249934994499549964997499849995000500150025003500450055006 |
- // Copyright 2015 The Chromium Authors. All rights reserved.
- // Use of this source code is governed by a BSD-style license that can be
- // found in the LICENSE file.
- #include <memory>
- #include <tuple>
- #include <utility>
- #include "base/bind.h"
- #include "base/containers/flat_map.h"
- #include "base/files/file_path.h"
- #include "base/files/file_util.h"
- #include "base/location.h"
- #include "base/memory/ref_counted.h"
- #include "base/path_service.h"
- #include "base/run_loop.h"
- #include "base/task/task_traits.h"
- #include "base/task/thread_pool.h"
- #include "base/test/scoped_path_override.h"
- #include "base/test/task_environment.h"
- #include "base/threading/thread_task_runner_handle.h"
- #include "base/values.h"
- #include "base/version.h"
- #include "build/build_config.h"
- #include "components/crx_file/crx_verifier.h"
- #include "components/prefs/testing_pref_service.h"
- #include "components/update_client/component_unpacker.h"
- #include "components/update_client/crx_downloader_factory.h"
- #include "components/update_client/crx_update_item.h"
- #include "components/update_client/network.h"
- #include "components/update_client/patcher.h"
- #include "components/update_client/persisted_data.h"
- #include "components/update_client/ping_manager.h"
- #include "components/update_client/protocol_handler.h"
- #include "components/update_client/test_configurator.h"
- #include "components/update_client/test_installer.h"
- #include "components/update_client/unzipper.h"
- #include "components/update_client/update_checker.h"
- #include "components/update_client/update_client_errors.h"
- #include "components/update_client/update_client_internal.h"
- #include "components/update_client/update_engine.h"
- #include "testing/gmock/include/gmock/gmock.h"
- #include "testing/gtest/include/gtest/gtest.h"
- #include "third_party/abseil-cpp/absl/types/optional.h"
- #include "url/gurl.h"
- namespace update_client {
- namespace {
- using base::FilePath;
- // Makes a copy of the file specified by |from_path| in a temporary directory
- // and returns the path of the copy. Returns true if successful. Cleans up if
- // there was an error creating the copy.
- bool MakeTestFile(const FilePath& from_path, FilePath* to_path) {
- FilePath temp_dir;
- bool result =
- CreateNewTempDirectory(FILE_PATH_LITERAL("update_client"), &temp_dir);
- if (!result)
- return false;
- FilePath temp_file;
- result = CreateTemporaryFileInDir(temp_dir, &temp_file);
- if (!result)
- return false;
- result = CopyFile(from_path, temp_file);
- if (!result) {
- base::DeleteFile(temp_file);
- return false;
- }
- *to_path = temp_file;
- return true;
- }
- using Events = UpdateClient::Observer::Events;
- class MockObserver : public UpdateClient::Observer {
- public:
- MOCK_METHOD2(OnEvent, void(Events event, const std::string&));
- };
- class MockActionHandler : public ActionHandler {
- public:
- MockActionHandler() = default;
- MockActionHandler(const MockActionHandler&) = delete;
- MockActionHandler& operator=(const MockActionHandler&) = delete;
- MOCK_METHOD3(Handle,
- void(const base::FilePath&, const std::string&, Callback));
- private:
- ~MockActionHandler() override = default;
- };
- class MockCrxStateChangeReceiver
- : public base::RefCountedThreadSafe<MockCrxStateChangeReceiver> {
- public:
- MOCK_METHOD(void, Receive, (CrxUpdateItem));
- private:
- friend class base::RefCountedThreadSafe<MockCrxStateChangeReceiver>;
- ~MockCrxStateChangeReceiver() = default;
- };
- class MockCrxDownloaderFactory : public CrxDownloaderFactory {
- public:
- explicit MockCrxDownloaderFactory(scoped_refptr<CrxDownloader> crx_downloader)
- : crx_downloader_(crx_downloader) {}
- private:
- ~MockCrxDownloaderFactory() override = default;
- // Overrides for CrxDownloaderFactory.
- scoped_refptr<CrxDownloader> MakeCrxDownloader(
- bool /* background_download_enabled */) const override {
- return crx_downloader_;
- }
- scoped_refptr<CrxDownloader> crx_downloader_;
- };
- } // namespace
- using ::testing::_;
- using ::testing::AnyNumber;
- using ::testing::AtLeast;
- using ::testing::DoAll;
- using ::testing::InSequence;
- using ::testing::Invoke;
- using ::testing::Mock;
- using ::testing::Return;
- using ::testing::Unused;
- using std::string;
- class MockPingManagerImpl : public PingManager {
- public:
- struct PingData {
- std::string id;
- base::Version previous_version;
- base::Version next_version;
- ErrorCategory error_category = ErrorCategory::kNone;
- int error_code = 0;
- int extra_code1 = 0;
- ErrorCategory diff_error_category = ErrorCategory::kNone;
- int diff_error_code = 0;
- bool diff_update_failed = false;
- };
- explicit MockPingManagerImpl(scoped_refptr<Configurator> config);
- MockPingManagerImpl(const MockPingManagerImpl&) = delete;
- MockPingManagerImpl& operator=(const MockPingManagerImpl&) = delete;
- void SendPing(const Component& component,
- const PersistedData& metadata,
- Callback callback) override;
- const std::vector<PingData>& ping_data() const;
- const std::vector<base::Value>& events() const;
- protected:
- ~MockPingManagerImpl() override;
- private:
- std::vector<PingData> ping_data_;
- std::vector<base::Value> events_;
- };
- MockPingManagerImpl::MockPingManagerImpl(scoped_refptr<Configurator> config)
- : PingManager(config) {}
- MockPingManagerImpl::~MockPingManagerImpl() = default;
- void MockPingManagerImpl::SendPing(const Component& component,
- const PersistedData& metadata,
- Callback callback) {
- PingData ping_data;
- ping_data.id = component.id_;
- ping_data.previous_version = component.previous_version_;
- ping_data.next_version = component.next_version_;
- ping_data.error_category = component.error_category_;
- ping_data.error_code = component.error_code_;
- ping_data.extra_code1 = component.extra_code1_;
- ping_data.diff_error_category = component.diff_error_category_;
- ping_data.diff_error_code = component.diff_error_code_;
- ping_data.diff_update_failed = component.diff_update_failed();
- ping_data_.push_back(ping_data);
- events_ = component.GetEvents();
- std::move(callback).Run(0, "");
- }
- const std::vector<MockPingManagerImpl::PingData>&
- MockPingManagerImpl::ping_data() const {
- return ping_data_;
- }
- const std::vector<base::Value>& MockPingManagerImpl::events() const {
- return events_;
- }
- class UpdateClientTest : public testing::Test {
- public:
- UpdateClientTest(const UpdateClientTest&) = delete;
- UpdateClientTest& operator=(const UpdateClientTest&) = delete;
- protected:
- UpdateClientTest();
- ~UpdateClientTest() override = default;
- void RunThreads();
- // Returns the full path to a test file.
- static base::FilePath TestFilePath(const char* file);
- scoped_refptr<update_client::TestConfigurator> config() { return config_; }
- update_client::PersistedData* metadata() { return metadata_.get(); }
- base::OnceClosure quit_closure() { return runloop_.QuitClosure(); }
- // Injects the CrxDownloaderFactory in the test fixture.
- template <typename MockCrxDownloaderT>
- void SetMockCrxDownloader() {
- config()->SetCrxDownloaderFactory(
- base::MakeRefCounted<MockCrxDownloaderFactory>(
- base::MakeRefCounted<MockCrxDownloaderT>()));
- }
- private:
- static constexpr int kNumWorkerThreads_ = 2;
- base::test::TaskEnvironment task_environment_;
- base::RunLoop runloop_;
- std::unique_ptr<TestingPrefServiceSimple> pref_ =
- std::make_unique<TestingPrefServiceSimple>();
- scoped_refptr<update_client::TestConfigurator> config_ =
- base::MakeRefCounted<TestConfigurator>(pref_.get());
- std::unique_ptr<update_client::PersistedData> metadata_ =
- std::make_unique<PersistedData>(pref_.get(), nullptr);
- };
- constexpr int UpdateClientTest::kNumWorkerThreads_;
- UpdateClientTest::UpdateClientTest() {
- PersistedData::RegisterPrefs(pref_->registry());
- }
- void UpdateClientTest::RunThreads() {
- runloop_.Run();
- task_environment_.RunUntilIdle();
- }
- base::FilePath UpdateClientTest::TestFilePath(const char* file) {
- base::FilePath path;
- base::PathService::Get(base::DIR_SOURCE_ROOT, &path);
- return path.AppendASCII("components")
- .AppendASCII("test")
- .AppendASCII("data")
- .AppendASCII("update_client")
- .AppendASCII(file);
- }
- // Tests the scenario where one update check is done for one CRX. The CRX
- // has no update.
- TEST_F(UpdateClientTest, OneCrxNoUpdate) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- std::vector<absl::optional<CrxComponent>> component = {crx};
- return component;
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- auto& component = context->components.at(id);
- EXPECT_TRUE(component->is_foreground());
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver), true,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(2u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpToDate, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the scenario where two CRXs are checked for updates. On CRX has
- // an update, the other CRX does not.
- TEST_F(UpdateClientTest, TwoCrxUpdateNoUpdate) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx1;
- crx1.name = "test_jebg";
- crx1.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx1.version = base::Version("0.9");
- crx1.installer = base::MakeRefCounted<TestInstaller>();
- crx1.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- CrxComponent crx2;
- crx2.name = "test_abag";
- crx2.pk_hash.assign(abag_hash, abag_hash + std::size(abag_hash));
- crx2.version = base::Version("2.2");
- crx2.installer = base::MakeRefCounted<TestInstaller>();
- crx2.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx1, crx2};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- <app appid='abagagagagagagagagagagagagagagag'>
- <updatecheck status='noupdate'/>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(2u, context->components_to_check_for_updates.size());
- ProtocolParser::Results results;
- {
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = "jebgalgnebhfojomionfpkfelancnnkf";
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- EXPECT_FALSE(context->components.at(id)->is_foreground());
- }
- {
- const std::string id = "abagagagagagagagagagagagagagagag";
- EXPECT_EQ(id, context->components_to_check_for_updates[1]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- results.list.push_back(result);
- EXPECT_FALSE(context->components.at(id)->is_foreground());
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- FilePath path;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- Result result;
- result.error = 0;
- result.response = path;
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes / 2,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "abagagagagagagagagagagagagagagag")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "abagagagagagagagagagagagagagagag"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "abagagagagagagagagagagagagagagag"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(9u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kChecking, items[1].state);
- EXPECT_STREQ("abagagagagagagagagagagagagagagag", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[6].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[7].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[7].id.c_str());
- EXPECT_EQ(ComponentState::kUpToDate, items[8].state);
- EXPECT_STREQ("abagagagagagagagagagagagagagagag", items[8].id.c_str());
- std::vector<std::tuple<int64_t, int64_t>> progress_bytes = {
- {-1, -1}, {-1, -1}, {-1, -1}, {-1, -1}, {921, 1843},
- {1843, 1843}, {1843, 1843}, {1843, 1843}, {-1, -1}};
- EXPECT_EQ(items.size(), progress_bytes.size());
- for (size_t i{0}; i != items.size(); ++i) {
- EXPECT_EQ(items[i].downloaded_bytes, std::get<0>(progress_bytes[i]));
- EXPECT_EQ(items[i].total_bytes, std::get<1>(progress_bytes[i]));
- }
- update_client->RemoveObserver(&observer);
- }
- // Tests the scenario where two CRXs are checked for updates. One CRX has
- // an update but the server ignores the second CRX and returns no response for
- // it. The second component gets an |UPDATE_RESPONSE_NOT_FOUND| error and
- // transitions to the error state.
- TEST_F(UpdateClientTest, TwoCrxUpdateFirstServerIgnoresSecond) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx1;
- crx1.name = "test_jebg";
- crx1.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx1.version = base::Version("0.9");
- crx1.installer = base::MakeRefCounted<TestInstaller>();
- crx1.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- CrxComponent crx2;
- crx2.name = "test_abag";
- crx2.pk_hash.assign(abag_hash, abag_hash + std::size(abag_hash));
- crx2.version = base::Version("2.2");
- crx2.installer = base::MakeRefCounted<TestInstaller>();
- crx2.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx1, crx2};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(2u, context->components_to_check_for_updates.size());
- ProtocolParser::Results results;
- {
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = "jebgalgnebhfojomionfpkfelancnnkf";
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- EXPECT_FALSE(context->components.at(id)->is_foreground());
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- FilePath path;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- Result result;
- result.error = 0;
- result.response = path;
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "abagagagagagagagagagagagagagagag"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "abagagagagagagagagagagagagagagag"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-10004, item.error_code);
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "abagagagagagagagagagagagagagagag"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(8u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kChecking, items[1].state);
- EXPECT_STREQ("abagagagagagagagagagagagagagagag", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[6].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[7].state);
- EXPECT_STREQ("abagagagagagagagagagagagagagagag", items[7].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the update check for two CRXs scenario when the second CRX does not
- // provide a CrxComponent instance. In this case, the update is handled as
- // if only one component were provided as an argument to the |Update| call
- // with the exception that the second component still fires an event such as
- // |COMPONENT_UPDATE_ERROR|.
- TEST_F(UpdateClientTest, TwoCrxUpdateNoCrxComponentData) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx, absl::nullopt};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- ProtocolParser::Results results;
- {
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- EXPECT_FALSE(context->components.at(id)->is_foreground());
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "ihfokbkgjpifnbbojhneepfflplebdkc"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(7u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[6].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[6].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the update check for two CRXs scenario when no CrxComponent data is
- // provided for either component. In this case, no update check occurs, and
- // |COMPONENT_UPDATE_ERROR| event fires for both components.
- TEST_F(UpdateClientTest, TwoCrxUpdateNoCrxComponentDataAtAll) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- return {absl::nullopt, absl::nullopt};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- NOTREACHED();
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { NOTREACHED(); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- EXPECT_EQ(0u, MockPingManagerImpl::ping_data().size());
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "ihfokbkgjpifnbbojhneepfflplebdkc"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(2u, items.size());
- EXPECT_EQ(ComponentState::kUpdateError, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the scenario where there is a download timeout for the first
- // CRX. The update for the first CRX fails. The update client waits before
- // attempting the update for the second CRX. This update succeeds.
- TEST_F(UpdateClientTest, TwoCrxUpdateDownloadTimeout) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx1;
- crx1.name = "test_jebg";
- crx1.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx1.version = base::Version("0.9");
- crx1.installer = base::MakeRefCounted<TestInstaller>();
- crx1.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- CrxComponent crx2;
- crx2.name = "test_ihfo";
- crx2.pk_hash.assign(ihfo_hash, ihfo_hash + std::size(ihfo_hash));
- crx2.version = base::Version("0.8");
- crx2.installer = base::MakeRefCounted<TestInstaller>();
- crx2.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx1, crx2};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_1.crx'
- hash_sha256='8f5aa190311237cae00675af87ff457f278cd1a05
- 895470ac5d46647d4a3c2ea'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(2u, context->components_to_check_for_updates.size());
- ProtocolParser::Results results;
- {
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- }
- {
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[1]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_1.crx";
- package.hash_sha256 =
- "8f5aa190311237cae00675af87ff457f278cd1a05895470ac5d46647d4a3c2ea";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = -118;
- download_metrics.downloaded_bytes = 0;
- download_metrics.total_bytes = 0;
- download_metrics.download_time_ms = 1000;
- // The result must not include a file path in the case of errors.
- result.error = -118;
- } else if (url.path() ==
- "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 53638;
- download_metrics.total_bytes = 53638;
- download_metrics.download_time_ms = 2000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(2u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(1, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(-118, ping_data[0].error_code);
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[1].id);
- EXPECT_EQ(base::Version("0.8"), ping_data[1].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[1].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[1].error_category));
- EXPECT_EQ(0, ping_data[1].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(1, static_cast<int>(item.error_category));
- EXPECT_EQ(-118, item.error_code);
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_WAIT,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AnyNumber());
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "ihfokbkgjpifnbbojhneepfflplebdkc"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(11u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kChecking, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[6].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[7].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[7].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[8].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[8].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[9].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[9].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[10].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[10].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the differential update scenario for one CRX. Tests install progress
- // for differential and full updates.
- TEST_F(UpdateClientTest, OneCrxDiffUpdate) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- static int num_calls = 0;
- // Must use the same stateful installer object.
- static auto installer = base::MakeRefCounted<VersionedTestInstaller>();
- installer->set_installer_progress_samples({-1, 50, 100});
- ++num_calls;
- CrxComponent crx;
- crx.name = "test_ihfo";
- crx.pk_hash.assign(ihfo_hash, ihfo_hash + std::size(ihfo_hash));
- crx.installer = installer;
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- if (num_calls == 1) {
- crx.version = base::Version("0.8");
- } else if (num_calls == 2) {
- crx.version = base::Version("1.0");
- } else {
- NOTREACHED();
- }
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- static int num_call = 0;
- ++num_call;
- ProtocolParser::Results results;
- if (num_call == 1) {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_1.crx'
- hash_sha256='813c59747e139a608b3b5fc49633affc6db57437
- 3f309f156ea6d27229c0b3f9'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_1.crx";
- package.hash_sha256 =
- "8f5aa190311237cae00675af87ff457f278cd1a05895470ac5d46647d4a3c2ea";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- } else if (num_call == 2) {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- <url codebasediff='http://localhost/download/'/>
- </urls>
- <manifest version='2.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_2.crx'
- namediff='ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx'
- hash_sha256='c87d8742c3ff3d7a0cb6f3c91aa2fcf3dea6361
- 8086a7db1c5be5300e1d4d6b6'
- fp='22'
- hashdiff_sha256='0fd48a5dd87006a709756cfc47198cbc4c4
- 928f33ac4277d79573c15164a33eb'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_2.crx";
- package.namediff = "ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx";
- package.hash_sha256 =
- "c87d8742c3ff3d7a0cb6f3c91aa2fcf3dea63618086a7db1c5be5300e1d4d6b6";
- package.hashdiff_sha256 =
- "0fd48a5dd87006a709756cfc47198cbc4c4928f33ac4277d79573c15164a33eb";
- package.fingerprint = "22";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.crx_diffurls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "2.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 53638;
- download_metrics.total_bytes = 53638;
- download_metrics.download_time_ms = 2000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1.crx"), &path));
- result.error = 0;
- result.response = path;
- } else if (url.path() ==
- "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 2105;
- download_metrics.total_bytes = 2105;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes / 2,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(2u, ping_data.size());
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[0].id);
- EXPECT_EQ(base::Version("0.8"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[1].id);
- EXPECT_EQ(base::Version("1.0"), ping_data[1].previous_version);
- EXPECT_EQ(base::Version("2.0"), ping_data[1].next_version);
- EXPECT_FALSE(ping_data[1].diff_update_failed);
- EXPECT_EQ(0, static_cast<int>(ping_data[1].diff_error_category));
- EXPECT_EQ(0, ping_data[1].diff_error_code);
- EXPECT_EQ(0, static_cast<int>(ping_data[1].error_category));
- EXPECT_EQ(0, ping_data[1].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_UPDATING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(3);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_UPDATING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(3);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- }
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"ihfokbkgjpifnbbojhneepfflplebdkc"};
- {
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- base::RunLoop runloop;
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- EXPECT_EQ(10u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[2].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[5].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[6].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[7].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[7].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[8].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[8].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[9].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[9].id.c_str());
- std::vector<int> samples = {-1, -1, -1, -1, -1, -1, -1, 50, 100, 100};
- EXPECT_EQ(items.size(), samples.size());
- for (size_t i = 0; i != items.size(); ++i)
- EXPECT_EQ(items[i].install_progress, samples[i]);
- }
- {
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- base::RunLoop runloop;
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- EXPECT_EQ(10u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloadingDiff, items[2].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloadingDiff, items[3].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloadingDiff, items[4].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdatingDiff, items[5].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdatingDiff, items[6].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdatingDiff, items[7].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[7].id.c_str());
- EXPECT_EQ(ComponentState::kUpdatingDiff, items[8].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[8].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[9].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[9].id.c_str());
- std::vector<int> samples = {-1, -1, -1, -1, -1, -1, -1, 50, 100, 100};
- EXPECT_EQ(items.size(), samples.size());
- for (size_t i = 0; i != items.size(); ++i)
- EXPECT_EQ(items[i].install_progress, samples[i]);
- }
- update_client->RemoveObserver(&observer);
- }
- // Tests the update scenario for one CRX where the CRX installer returns
- // an error. Tests that the |unpack_path| argument refers to a valid path
- // then |Install| is called, then tests that the |unpack| path is deleted
- // by the |update_client| code before the test ends.
- TEST_F(UpdateClientTest, OneCrxInstallError) {
- class MockInstaller : public CrxInstaller {
- public:
- MOCK_METHOD1(OnUpdateError, void(int error));
- MOCK_METHOD1(DoInstall, void(const base::FilePath& unpack_path));
- MOCK_METHOD2(GetInstalledFile,
- bool(const std::string& file, base::FilePath* installed_file));
- MOCK_METHOD0(Uninstall, bool());
- void Install(const base::FilePath& unpack_path,
- const std::string& public_key,
- std::unique_ptr<InstallParams> /*install_params*/,
- ProgressCallback progress_callback,
- Callback callback) override {
- DoInstall(unpack_path);
- unpack_path_ = unpack_path;
- EXPECT_TRUE(base::DirectoryExists(unpack_path_));
- base::ThreadPool::PostTask(
- FROM_HERE, {base::MayBlock()},
- base::BindOnce(std::move(callback),
- CrxInstaller::Result(InstallError::GENERIC_ERROR)));
- }
- protected:
- ~MockInstaller() override {
- // The unpack path is deleted unconditionally by the component state code,
- // which is driving this installer. Therefore, the unpack path must not
- // exist when this object is destroyed.
- if (!unpack_path_.empty())
- EXPECT_FALSE(base::DirectoryExists(unpack_path_));
- }
- private:
- // Contains the |unpack_path| argument of the Install call.
- base::FilePath unpack_path_;
- };
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- scoped_refptr<MockInstaller> installer =
- base::MakeRefCounted<MockInstaller>();
- EXPECT_CALL(*installer, OnUpdateError(_)).Times(0);
- EXPECT_CALL(*installer, DoInstall(_)).Times(1);
- EXPECT_CALL(*installer, GetInstalledFile(_, _)).Times(0);
- EXPECT_CALL(*installer, Uninstall()).Times(0);
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = installer;
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- FilePath path;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- Result result;
- result.error = 0;
- result.response = path;
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(3, static_cast<int>(ping_data[0].error_category)); // kInstall.
- EXPECT_EQ(9, ping_data[0].error_code); // kInstallerError.
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "jebgalgnebhfojomionfpkfelancnnkf")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(6u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the fallback from differential to full update scenario for one CRX.
- TEST_F(UpdateClientTest, OneCrxDiffUpdateFailsFullUpdateSucceeds) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- static int num_calls = 0;
- // Must use the same stateful installer object.
- static auto installer = base::MakeRefCounted<VersionedTestInstaller>();
- ++num_calls;
- CrxComponent crx;
- crx.name = "test_ihfo";
- crx.pk_hash.assign(ihfo_hash, ihfo_hash + std::size(ihfo_hash));
- crx.installer = installer;
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- if (num_calls == 1) {
- crx.version = base::Version("0.8");
- } else if (num_calls == 2) {
- crx.version = base::Version("1.0");
- } else {
- NOTREACHED();
- }
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- static int num_call = 0;
- ++num_call;
- ProtocolParser::Results results;
- if (num_call == 1) {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_1.crx'
- hash_sha256='813c59747e139a608b3b5fc49633affc6db57437
- 3f309f156ea6d27229c0b3f9'
- fp='1'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_1.crx";
- package.hash_sha256 =
- "8f5aa190311237cae00675af87ff457f278cd1a05895470ac5d46647d4a3c2ea";
- package.fingerprint = "1";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- } else if (num_call == 2) {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- <url codebasediff='http://localhost/download/'/>
- </urls>
- <manifest version='2.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_2.crx'
- namediff='ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx'
- hash_sha256='c87d8742c3ff3d7a0cb6f3c91aa2fcf3dea6361
- 8086a7db1c5be5300e1d4d6b6'
- fp='22'
- hashdiff_sha256='0fd48a5dd87006a709756cfc47198cbc4c4
- 928f33ac4277d79573c15164a33eb'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_2.crx";
- package.namediff = "ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx";
- package.hash_sha256 =
- "c87d8742c3ff3d7a0cb6f3c91aa2fcf3dea63618086a7db1c5be5300e1d4d6b6";
- package.hashdiff_sha256 =
- "0fd48a5dd87006a709756cfc47198cbc4c4928f33ac4277d79573c15164a33eb";
- package.fingerprint = "22";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.crx_diffurls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "2.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 53638;
- download_metrics.total_bytes = 53638;
- download_metrics.download_time_ms = 2000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1.crx"), &path));
- result.error = 0;
- result.response = path;
- } else if (url.path() ==
- "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx") {
- // A download error is injected on this execution path.
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = -1;
- download_metrics.downloaded_bytes = 0;
- download_metrics.total_bytes = 2105;
- download_metrics.download_time_ms = 1000;
- // The response must not include a file path in the case of errors.
- result.error = -1;
- } else if (url.path() ==
- "/download/ihfokbkgjpifnbbojhneepfflplebdkc_2.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 53855;
- download_metrics.total_bytes = 53855;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_2.crx"), &path));
- result.error = 0;
- result.response = path;
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(2u, ping_data.size());
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[0].id);
- EXPECT_EQ(base::Version("0.8"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[1].id);
- EXPECT_EQ(base::Version("1.0"), ping_data[1].previous_version);
- EXPECT_EQ(base::Version("2.0"), ping_data[1].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[1].error_category));
- EXPECT_EQ(0, ping_data[1].error_code);
- EXPECT_TRUE(ping_data[1].diff_update_failed);
- EXPECT_EQ(1, static_cast<int>(ping_data[1].diff_error_category));
- EXPECT_EQ(-1, ping_data[1].diff_error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc")).Times(1);
- }
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"ihfokbkgjpifnbbojhneepfflplebdkc"};
- {
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- base::RunLoop runloop;
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- EXPECT_EQ(6u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[2].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[4].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[5].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[5].id.c_str());
- }
- {
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- base::RunLoop runloop;
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- EXPECT_EQ(8u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloadingDiff, items[2].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloadingDiff, items[3].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[4].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[5].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[6].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[7].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[7].id.c_str());
- }
- update_client->RemoveObserver(&observer);
- }
- // Tests the queuing of update checks. In this scenario, two update checks are
- // done for one CRX. The second update check call is queued up and will run
- // after the first check has completed. The CRX has no updates.
- TEST_F(UpdateClientTest, OneCrxNoUpdateQueuedCall) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- static int num_call = 0;
- ++num_call;
- EXPECT_EQ(Error::NONE, error);
- if (num_call == 2)
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- auto& component = context->components.at(id);
- EXPECT_FALSE(component->is_foreground());
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items1;
- auto receiver1 = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver1, Receive(_))
- .WillRepeatedly(
- [&items1](const CrxUpdateItem& item) { items1.push_back(item); });
- std::vector<CrxUpdateItem> items2;
- auto receiver2 = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver2, Receive(_))
- .WillRepeatedly(
- [&items2](const CrxUpdateItem& item) { items2.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver1),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver2),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(2u, items1.size());
- EXPECT_EQ(ComponentState::kChecking, items1[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items1[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpToDate, items1[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items1[1].id.c_str());
- EXPECT_EQ(2u, items2.size());
- EXPECT_EQ(ComponentState::kChecking, items2[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items2[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpToDate, items2[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items2[1].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the install of one CRX. Tests the installer is invoked with the
- // run and arguments values of the manifest object. Tests that "pv" and "fp"
- // are persisted.
- TEST_F(UpdateClientTest, OneCrxInstall) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'
- run='UpdaterSetup.exe' arguments='--arg1 --arg2'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- package.fingerprint = "some-fingerprint";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.run = "UpdaterSetup.exe";
- result.manifest.arguments = "--arg1 --arg2";
- result.manifest.packages.push_back(package);
- ProtocolParser::Results results;
- results.list.push_back(result);
- // Verify that calling Install sets ondemand.
- EXPECT_TRUE(context->components.at(id)->is_foreground());
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.0"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(0, ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- {
- EXPECT_FALSE(config()->GetPrefService()->FindPreference(
- "updateclientdata.apps.jebgalgnebhfojomionfpkfelancnnkf.pv"));
- EXPECT_FALSE(config()->GetPrefService()->FindPreference(
- "updateclientdata.apps.jebgalgnebhfojomionfpkfelancnnkf.fp"));
- }
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([update_client](Events event, const std::string& id) {
- CrxUpdateItem update_item;
- ASSERT_TRUE(update_client->GetCrxUpdateState(id, &update_item));
- ASSERT_TRUE(update_item.component);
- const auto* test_installer = static_cast<TestInstaller*>(
- update_item.component->installer.get());
- EXPECT_STREQ("UpdaterSetup.exe",
- test_installer->install_params()->run.c_str());
- EXPECT_STREQ("--arg1 --arg2",
- test_installer->install_params()->arguments.c_str());
- }));
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(6u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[5].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[5].id.c_str());
- const base::Value::Dict& dict =
- config()->GetPrefService()->GetValueDict("updateclientdata");
- const std::string* pv =
- dict.FindStringByDottedPath("apps.jebgalgnebhfojomionfpkfelancnnkf.pv");
- ASSERT_TRUE(pv);
- EXPECT_STREQ("1.0", pv->c_str());
- const std::string* fingerprint =
- dict.FindStringByDottedPath("apps.jebgalgnebhfojomionfpkfelancnnkf.fp");
- ASSERT_TRUE(fingerprint);
- EXPECT_STREQ("some-fingerprint", fingerprint->c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the install of one CRX when no component data is provided. This
- // results in an install error.
- TEST_F(UpdateClientTest, OneCrxInstallNoCrxComponentData) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- return {absl::nullopt};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- NOTREACHED();
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { NOTREACHED(); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- EXPECT_EQ(0u, MockPingManagerImpl::ping_data().size());
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- // Tests that the state of the component when the CrxComponent data
- // is not provided. In this case, the optional |item.component|
- // instance is not present.
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", item.id.c_str());
- EXPECT_FALSE(item.component);
- EXPECT_EQ(ErrorCategory::kService, item.error_category);
- EXPECT_EQ(static_cast<int>(Error::CRX_NOT_FOUND), item.error_code);
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(1u, items.size());
- EXPECT_EQ(ComponentState::kUpdateError, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests that overlapping installs of the same CRX result in an error.
- TEST_F(UpdateClientTest, ConcurrentInstallSameCRX) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- static int num_call = 0;
- ++num_call;
- EXPECT_LE(num_call, 2);
- if (num_call == 1) {
- EXPECT_EQ(Error::UPDATE_IN_PROGRESS, error);
- return;
- }
- if (num_call == 2) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- ProtocolParser::Results results;
- results.list.push_back(result);
- // Verify that calling Install sets |is_foreground| for the component.
- EXPECT_TRUE(context->components.at(id)->is_foreground());
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- std::vector<CrxUpdateItem> items1;
- auto receiver1 = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver1, Receive(_))
- .WillRepeatedly(
- [&items1](const CrxUpdateItem& item) { items1.push_back(item); });
- std::vector<CrxUpdateItem> items2;
- auto receiver2 = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver2, Receive(_))
- .WillRepeatedly(
- [&items2](const CrxUpdateItem& item) { items2.push_back(item); });
- update_client->AddObserver(&observer);
- update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver1),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver2),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(2u, items1.size());
- EXPECT_EQ(ComponentState::kChecking, items1[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items1[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpToDate, items1[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items1[1].id.c_str());
- EXPECT_TRUE(items2.empty());
- update_client->RemoveObserver(&observer);
- }
- // Tests that UpdateClient::Update returns Error::INVALID_ARGUMENT when
- // the |ids| parameter is empty.
- TEST_F(UpdateClientTest, EmptyIdList) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- return {};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::INVALID_ARGUMENT, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- NOTREACHED();
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- const std::vector<std::string> empty_id_list;
- update_client->Update(
- empty_id_list, base::BindOnce(&DataCallbackMock::Callback), {}, false,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- }
- TEST_F(UpdateClientTest, SendUninstallPing) {
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return nullptr;
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- NOTREACHED();
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- static scoped_refptr<CrxDownloader> Create(
- bool is_background_download,
- scoped_refptr<NetworkFetcherFactory> network_fetcher_factory) {
- return nullptr;
- }
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {}
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("1.2.3.4"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("0"), ping_data[0].next_version);
- EXPECT_EQ(10, ping_data[0].extra_code1);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- CrxComponent crx;
- crx.app_id = "jebgalgnebhfojomionfpkfelancnnkf";
- crx.version = base::Version("1.2.3.4");
- update_client->SendUninstallPing(
- crx, 10,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- }
- TEST_F(UpdateClientTest, RetryAfter) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- static int num_call = 0;
- ++num_call;
- EXPECT_LE(num_call, 4);
- if (num_call == 1) {
- EXPECT_EQ(Error::NONE, error);
- } else if (num_call == 2) {
- // This request is throttled since the update engine received a
- // positive |retry_after_sec| value in the update check response.
- EXPECT_EQ(Error::RETRY_LATER, error);
- } else if (num_call == 3) {
- // This request is a foreground Install, which is never throttled.
- // The update engine received a |retry_after_sec| value of 0, which
- // resets the throttling.
- EXPECT_EQ(Error::NONE, error);
- } else if (num_call == 4) {
- // This request succeeds since there is no throttling in effect.
- EXPECT_EQ(Error::NONE, error);
- }
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- static int num_call = 0;
- ++num_call;
- EXPECT_LE(num_call, 3);
- int retry_after_sec(0);
- if (num_call == 1) {
- // Throttle the next call.
- retry_after_sec = 60 * 60; // 1 hour.
- }
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, retry_after_sec));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_ALREADY_UP_TO_DATE,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- {
- // The engine handles this Update call but responds with a valid
- // |retry_after_sec|, which causes subsequent calls to fail.
- base::RunLoop runloop;
- update_client->Update(ids, base::BindOnce(&DataCallbackMock::Callback), {},
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- }
- {
- // This call will result in a completion callback invoked with
- // Error::ERROR_UPDATE_RETRY_LATER.
- base::RunLoop runloop;
- update_client->Update(ids, base::BindOnce(&DataCallbackMock::Callback), {},
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- }
- {
- // The Install call is handled, and the throttling is reset due to
- // the value of |retry_after_sec| in the completion callback.
- base::RunLoop runloop;
- update_client->Install(std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback), {},
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- }
- {
- // This call succeeds.
- base::RunLoop runloop;
- update_client->Update(ids, base::BindOnce(&DataCallbackMock::Callback), {},
- false,
- base::BindOnce(&CompletionCallbackMock::Callback,
- runloop.QuitClosure()));
- runloop.Run();
- }
- update_client->RemoveObserver(&observer);
- }
- // Tests the update check for two CRXs scenario. The first component supports
- // the group policy to enable updates, and has its updates disabled. The second
- // component has an update. The server does not honor the "updatedisabled"
- // attribute and returns updates for both components. However, the update for
- // the first component is not applied and the client responds with a
- // (SERVICE_ERROR, UPDATE_DISABLED)
- TEST_F(UpdateClientTest, TwoCrxUpdateOneUpdateDisabled) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx1;
- crx1.name = "test_jebg";
- crx1.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx1.version = base::Version("0.9");
- crx1.installer = base::MakeRefCounted<TestInstaller>();
- crx1.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- crx1.updates_enabled = false;
- CrxComponent crx2;
- crx2.name = "test_ihfo";
- crx2.pk_hash.assign(ihfo_hash, ihfo_hash + std::size(ihfo_hash));
- crx2.version = base::Version("0.8");
- crx2.installer = base::MakeRefCounted<TestInstaller>();
- crx2.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx1, crx2};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='jebgalgnebhfojomionfpkfelancnnkf'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='jebgalgnebhfojomionfpkfelancnnkf.crx'
- hash_sha256='7ab32f071cd9b5ef8e0d7913be161f532d98b3e9f
- a284a7cd8059c3409ce0498'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- <app appid='ihfokbkgjpifnbbojhneepfflplebdkc'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='ihfokbkgjpifnbbojhneepfflplebdkc_1.crx'
- hash_sha256='8f5aa190311237cae00675af87ff457f278cd1a05
- 895470ac5d46647d4a3c2ea'/>
- </packages>
- </manifest>
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(2u, context->components_to_check_for_updates.size());
- ProtocolParser::Results results;
- {
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- }
- {
- const std::string id = "ihfokbkgjpifnbbojhneepfflplebdkc";
- EXPECT_EQ(id, context->components_to_check_for_updates[1]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "ihfokbkgjpifnbbojhneepfflplebdkc_1.crx";
- package.hash_sha256 =
- "8f5aa190311237cae00675af87ff457f278cd1a05895470ac5d46647d4a3c2ea";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- results.list.push_back(result);
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/ihfokbkgjpifnbbojhneepfflplebdkc_1.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 53638;
- download_metrics.total_bytes = 53638;
- download_metrics.download_time_ms = 2000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(2u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.9"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(4, static_cast<int>(ping_data[0].error_category));
- EXPECT_EQ(2, ping_data[0].error_code);
- EXPECT_EQ("ihfokbkgjpifnbbojhneepfflplebdkc", ping_data[1].id);
- EXPECT_EQ(base::Version("0.8"), ping_data[1].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[1].next_version);
- EXPECT_EQ(0, static_cast<int>(ping_data[1].error_category));
- EXPECT_EQ(0, ping_data[1].error_code);
- }
- };
- // Disables updates for the components declaring support for the group policy.
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(AtLeast(1));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_READY,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATED,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "ihfokbkgjpifnbbojhneepfflplebdkc"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(9u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kChecking, items[1].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[4].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[4].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[5].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[5].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[6].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[6].id.c_str());
- EXPECT_EQ(ComponentState::kUpdating, items[7].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[7].id.c_str());
- EXPECT_EQ(ComponentState::kUpdated, items[8].state);
- EXPECT_STREQ("ihfokbkgjpifnbbojhneepfflplebdkc", items[8].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the scenario where the update check fails.
- TEST_F(UpdateClientTest, OneCrxUpdateCheckFails) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::UPDATE_CHECK_ERROR, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE,
- base::BindOnce(std::move(update_check_callback), absl::nullopt,
- ErrorCategory::kUpdateCheck, -1, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-1, item.error_code);
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- false, base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(2u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests the scenario where the server responds with different values for
- // application status.
- TEST_F(UpdateClientTest, OneCrxErrorUnknownApp) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- std::vector<absl::optional<CrxComponent>> component;
- {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- component.push_back(crx);
- }
- {
- CrxComponent crx;
- crx.name = "test_abag";
- crx.pk_hash.assign(abag_hash, abag_hash + std::size(abag_hash));
- crx.version = base::Version("0.1");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- component.push_back(crx);
- }
- {
- CrxComponent crx;
- crx.name = "test_ihfo";
- crx.pk_hash.assign(ihfo_hash, ihfo_hash + std::size(ihfo_hash));
- crx.version = base::Version("0.2");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- component.push_back(crx);
- }
- {
- CrxComponent crx;
- crx.name = "test_gjpm";
- crx.pk_hash.assign(gjpm_hash, gjpm_hash + std::size(gjpm_hash));
- crx.version = base::Version("0.3");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- component.push_back(crx);
- }
- return component;
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(4u, context->components_to_check_for_updates.size());
- const std::string update_response =
- ")]}'"
- R"({"response": {)"
- R"( "protocol": "3.1",)"
- R"( "app": [)"
- R"({"appid": "jebgalgnebhfojomionfpkfelancnnkf",)"
- R"( "status": "error-unknownApplication"},)"
- R"({"appid": "abagagagagagagagagagagagagagagag",)"
- R"( "status": "restricted"},)"
- R"({"appid": "ihfokbkgjpifnbbojhneepfflplebdkc",)"
- R"( "status": "error-invalidAppId"},)"
- R"({"appid": "gjpmebpgbhcamgdgjcmnjfhggjpgcimm",)"
- R"( "status": "error-foobarApp"})"
- R"(]}})";
- const auto parser = ProtocolHandlerFactoryJSON().CreateParser();
- EXPECT_TRUE(parser->Parse(update_response));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE,
- base::BindOnce(std::move(update_check_callback), parser->results(),
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-10006, item.error_code); // UNKNOWN_APPPLICATION.
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "abagagagagagagagagagagagagagagag"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "abagagagagagagagagagagagagagagag"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-10007, item.error_code); // RESTRICTED_APPLICATION.
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "ihfokbkgjpifnbbojhneepfflplebdkc"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-10008, item.error_code); // INVALID_APPID.
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "gjpmebpgbhcamgdgjcmnjfhggjpgcimm"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "gjpmebpgbhcamgdgjcmnjfhggjpgcimm"))
- .Times(1)
- .WillOnce(Invoke([&update_client](Events event, const std::string& id) {
- CrxUpdateItem item;
- EXPECT_TRUE(update_client->GetCrxUpdateState(id, &item));
- EXPECT_EQ(ComponentState::kUpdateError, item.state);
- EXPECT_EQ(5, static_cast<int>(item.error_category));
- EXPECT_EQ(-10004, item.error_code); // UPDATE_RESPONSE_NOT_FOUND.
- EXPECT_EQ(0, item.extra_code1);
- }));
- }
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {
- "jebgalgnebhfojomionfpkfelancnnkf", "abagagagagagagagagagagagagagagag",
- "ihfokbkgjpifnbbojhneepfflplebdkc", "gjpmebpgbhcamgdgjcmnjfhggjpgcimm"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback), {}, true,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- update_client->RemoveObserver(&observer);
- }
- // Tests that a run action in invoked in the CRX install scenario.
- TEST_F(UpdateClientTest, ActionRun_Install) {
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='gjpmebpgbhcamgdgjcmnjfhggjpgcimm'>
- <updatecheck status='ok'>
- <urls>
- <url codebase='http://localhost/download/'/>
- </urls>
- <manifest version='1.0' prodversionmin='11.0.1.0'>
- <packages>
- <package name='runaction_test_win.crx3'
- hash_sha256='89290a0d2ff21ca5b45e109c6cc859ab5fe294e19c102d54acd321429c372cea'/>
- </packages>
- </manifest>
- <actions>"
- <action run='ChromeRecovery.crx3'/>"
- </actions>"
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "gjpmebpgbhcamgdgjcmnjfhggjpgcimm";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "runaction_test_win.crx3";
- package.hash_sha256 =
- "89290a0d2ff21ca5b45e109c6cc859ab5fe294e19c102d54acd321429c372cea";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.push_back(GURL("http://localhost/download/"));
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.packages.push_back(package);
- result.action_run = "ChromeRecovery.crx3";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/runaction_test_win.crx3") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(
- MakeTestFile(TestFilePath("runaction_test_win.crx3"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- EXPECT_EQ(3u, events().size());
- /*
- "<event eventtype="14" eventresult="1" downloader="unknown" "
- "url="http://localhost/download/runaction_test_win.crx3"
- "downloaded=1843 "
- "total=1843 download_time_ms="1000" previousversion="0.0" "
- "nextversion="1.0"/>"
- */
- const auto& event0 = events()[0];
- EXPECT_EQ(14, event0.FindKey("eventtype")->GetInt());
- EXPECT_EQ(1, event0.FindKey("eventresult")->GetInt());
- EXPECT_EQ("unknown", event0.FindKey("downloader")->GetString());
- EXPECT_EQ("http://localhost/download/runaction_test_win.crx3",
- event0.FindKey("url")->GetString());
- EXPECT_EQ(1843, event0.FindKey("downloaded")->GetDouble());
- EXPECT_EQ(1843, event0.FindKey("total")->GetDouble());
- EXPECT_EQ(1000, event0.FindKey("download_time_ms")->GetDouble());
- EXPECT_EQ("0.0", event0.FindKey("previousversion")->GetString());
- EXPECT_EQ("1.0", event0.FindKey("nextversion")->GetString());
- // "<event eventtype="42" eventresult="1" errorcode="1877345072"/>"
- const auto& event1 = events()[1];
- EXPECT_EQ(42, event1.FindKey("eventtype")->GetInt());
- EXPECT_EQ(1, event1.FindKey("eventresult")->GetInt());
- EXPECT_EQ(1877345072, event1.FindKey("errorcode")->GetInt());
- // "<event eventtype=\"2\" eventresult=\"1\" previousversion=\"0.0\" "
- // "nextversion=\"1.0\"/>",
- const auto& event2 = events()[2];
- EXPECT_EQ(2, event2.FindKey("eventtype")->GetInt());
- EXPECT_EQ(1, event1.FindKey("eventresult")->GetInt());
- EXPECT_EQ("0.0", event0.FindKey("previousversion")->GetString());
- EXPECT_EQ("1.0", event0.FindKey("nextversion")->GetString());
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- update_client->Install(
- std::string("gjpmebpgbhcamgdgjcmnjfhggjpgcimm"),
- base::BindOnce([](const std::vector<std::string>& ids) {
- auto action_handler = base::MakeRefCounted<MockActionHandler>();
- EXPECT_CALL(*action_handler, Handle(_, _, _))
- .WillOnce([](const base::FilePath& action,
- const std::string& session_id,
- ActionHandler::Callback callback) {
- EXPECT_EQ("ChromeRecovery.crx3",
- action.BaseName().MaybeAsASCII());
- EXPECT_TRUE(!session_id.empty());
- std::move(callback).Run(true, 1877345072, 0);
- });
- CrxComponent crx;
- crx.name = "test_niea";
- crx.pk_hash.assign(gjpm_hash, gjpm_hash + std::size(gjpm_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<VersionedTestInstaller>();
- crx.action_handler = action_handler;
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return std::vector<absl::optional<CrxComponent>>{crx};
- }),
- {},
- base::BindOnce(
- [](base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- },
- quit_closure()));
- RunThreads();
- }
- // Tests that a run action is invoked in an update scenario when there was
- // no update.
- TEST_F(UpdateClientTest, ActionRun_NoUpdate) {
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- /*
- Mock the following response:
- <?xml version='1.0' encoding='UTF-8'?>
- <response protocol='3.1'>
- <app appid='gjpmebpgbhcamgdgjcmnjfhggjpgcimm'>
- <updatecheck status='noupdate'>
- <actions>"
- <action run=ChromeRecovery.crx3'/>"
- </actions>"
- </updatecheck>
- </app>
- </response>
- */
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "gjpmebpgbhcamgdgjcmnjfhggjpgcimm";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- result.action_run = "ChromeRecovery.crx3";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- EXPECT_EQ(1u, events().size());
- // "<event eventtype="42" eventresult="1" errorcode="1877345072"/>"
- const auto& event = events()[0];
- EXPECT_EQ(42, event.FindKey("eventtype")->GetInt());
- EXPECT_EQ(1, event.FindKey("eventresult")->GetInt());
- EXPECT_EQ(1877345072, event.FindKey("errorcode")->GetInt());
- }
- };
- // Unpack the CRX to mock an existing install to be updated. The action to
- // run is going to be resolved relative to this directory.
- base::FilePath unpack_path;
- {
- base::RunLoop runloop;
- base::OnceClosure quit_closure = runloop.QuitClosure();
- auto config = base::MakeRefCounted<TestConfigurator>();
- auto component_unpacker = base::MakeRefCounted<ComponentUnpacker>(
- std::vector<uint8_t>(std::begin(gjpm_hash), std::end(gjpm_hash)),
- TestFilePath("runaction_test_win.crx3"), nullptr,
- config->GetUnzipperFactory()->Create(),
- config->GetPatcherFactory()->Create(), crx_file::VerifierFormat::CRX3);
- component_unpacker->Unpack(base::BindOnce(
- [](base::FilePath* unpack_path, base::OnceClosure quit_closure,
- const ComponentUnpacker::Result& result) {
- EXPECT_EQ(UnpackerError::kNone, result.error);
- EXPECT_EQ(0, result.extended_error);
- *unpack_path = result.unpack_path;
- std::move(quit_closure).Run();
- },
- &unpack_path, runloop.QuitClosure()));
- runloop.Run();
- }
- EXPECT_FALSE(unpack_path.empty());
- EXPECT_TRUE(base::DirectoryExists(unpack_path));
- int64_t file_size = 0;
- EXPECT_TRUE(base::GetFileSize(unpack_path.AppendASCII("ChromeRecovery.crx3"),
- &file_size));
- EXPECT_EQ(44582, file_size);
- base::ScopedTempDir unpack_path_owner;
- EXPECT_TRUE(unpack_path_owner.Set(unpack_path));
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- const std::vector<std::string> ids = {"gjpmebpgbhcamgdgjcmnjfhggjpgcimm"};
- update_client->Update(
- ids,
- base::BindOnce(
- [](const base::FilePath& unpack_path,
- const std::vector<std::string>& ids) {
- auto action_handler = base::MakeRefCounted<MockActionHandler>();
- EXPECT_CALL(*action_handler, Handle(_, _, _))
- .WillOnce([](const base::FilePath& action,
- const std::string& session_id,
- ActionHandler::Callback callback) {
- EXPECT_EQ("ChromeRecovery.crx3",
- action.BaseName().MaybeAsASCII());
- EXPECT_TRUE(!session_id.empty());
- std::move(callback).Run(true, 1877345072, 0);
- });
- CrxComponent crx;
- crx.name = "test_niea";
- crx.pk_hash.assign(gjpm_hash, gjpm_hash + std::size(gjpm_hash));
- crx.version = base::Version("1.0");
- crx.installer =
- base::MakeRefCounted<ReadOnlyTestInstaller>(unpack_path);
- crx.action_handler = action_handler;
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return std::vector<absl::optional<CrxComponent>>{crx};
- },
- unpack_path),
- {}, false,
- base::BindOnce(
- [](base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- },
- quit_closure()));
- RunThreads();
- }
- // Tests that custom response attributes are visible to observers.
- TEST_F(UpdateClientTest, CustomAttributeNoUpdate) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.9");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- std::vector<absl::optional<CrxComponent>> component = {crx};
- return component;
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_FALSE(context->session_id.empty());
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates.front());
- EXPECT_EQ(1u, context->components.count(id));
- auto& component = context->components.at(id);
- EXPECT_TRUE(component->is_foreground());
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "noupdate";
- result.custom_attributes["_example"] = "example_value";
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override { EXPECT_TRUE(false); }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- class Observer : public UpdateClient::Observer {
- public:
- explicit Observer(scoped_refptr<UpdateClient> update_client)
- : update_client_(update_client) {}
- void OnEvent(Events event, const std::string& id) override {
- if (event != Events::COMPONENT_ALREADY_UP_TO_DATE)
- return;
- ++calls;
- CrxUpdateItem item;
- EXPECT_TRUE(update_client_->GetCrxUpdateState(
- "jebgalgnebhfojomionfpkfelancnnkf", &item));
- EXPECT_EQ("example_value", item.custom_updatecheck_data["_example"]);
- }
- int calls = 0;
- private:
- scoped_refptr<UpdateClient> update_client_;
- };
- Observer observer(update_client);
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf"};
- update_client->Update(
- ids, base::BindOnce(&DataCallbackMock::Callback), {}, true,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- update_client->RemoveObserver(&observer);
- EXPECT_EQ(1, observer.calls);
- }
- // Tests the scenario where `CrxDataCallback` returns a vector whose elements
- // don't include a value for one of the component ids specified by the `ids`
- // parameter of the `UpdateClient::Update` function. Expects the completion
- // callback to include a specific error, and no other events and pings be
- // generated, since the update engine rejects the UpdateClient::Update call.
- TEST_F(UpdateClientTest, BadCrxDataCallback) {
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::BAD_CRX_DATA_CALLBACK, error);
- std::move(quit_closure).Run();
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override { EXPECT_TRUE(ping_data().empty()); }
- };
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- UpdateChecker::Factory{});
- MockObserver observer;
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- const std::vector<std::string> ids = {"jebgalgnebhfojomionfpkfelancnnkf",
- "gjpmebpgbhcamgdgjcmnjfhggjpgcimm"};
- // The `CrxDataCallback` argument only returns a value for the first
- // component id. This means that its result is ill formed, and the `Update`
- // call completes with an error.
- update_client->Update(
- ids, base::BindOnce([](const std::vector<std::string>& ids) {
- EXPECT_EQ(ids.size(), size_t{2});
- return std::vector<absl::optional<CrxComponent>>{absl::nullopt};
- }),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver), true,
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_TRUE(items.empty());
- update_client->RemoveObserver(&observer);
- }
- // Tests cancellation of an install before the task is run.
- TEST_F(UpdateClientTest, CancelInstallBeforeTaskStart) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::UPDATE_CANCELED, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- package.fingerprint = "some-fingerprint";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.emplace_back("http://localhost/download/");
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.run = "UpdaterSetup.exe";
- result.manifest.arguments = "--arg1 --arg2";
- result.manifest.packages.push_back(package);
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(0u, ping_data.size());
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client
- ->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()))
- .Run();
- RunThreads();
- EXPECT_EQ(0u, items.size());
- }
- // Tests cancellation of an install before the component installer runs.
- TEST_F(UpdateClientTest, CancelInstallBeforeInstall) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- package.fingerprint = "some-fingerprint";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.emplace_back("http://localhost/download/");
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.run = "UpdaterSetup.exe";
- result.manifest.arguments = "--arg1 --arg2";
- result.manifest.packages.push_back(package);
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.0"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(ErrorCategory::kService, ping_data[0].error_category);
- EXPECT_EQ(static_cast<int>(ServiceError::CANCELLED),
- ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- base::RepeatingClosure cancel;
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_DOWNLOADING,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(AtLeast(1))
- .WillRepeatedly(Invoke([&cancel]() { cancel.Run(); }));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- cancel = update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(5u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- EXPECT_EQ(ComponentState::kDownloading, items[3].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[3].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[4].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[4].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- // Tests cancellation of an install before the download.
- TEST_F(UpdateClientTest, CancelInstallBeforeDownload) {
- class DataCallbackMock {
- public:
- static std::vector<absl::optional<CrxComponent>> Callback(
- const std::vector<std::string>& ids) {
- CrxComponent crx;
- crx.name = "test_jebg";
- crx.pk_hash.assign(jebg_hash, jebg_hash + std::size(jebg_hash));
- crx.version = base::Version("0.0");
- crx.installer = base::MakeRefCounted<TestInstaller>();
- crx.crx_format_requirement = crx_file::VerifierFormat::CRX3;
- return {crx};
- }
- };
- class CompletionCallbackMock {
- public:
- static void Callback(base::OnceClosure quit_closure, Error error) {
- EXPECT_EQ(Error::NONE, error);
- std::move(quit_closure).Run();
- }
- };
- class MockUpdateChecker : public UpdateChecker {
- public:
- static std::unique_ptr<UpdateChecker> Create(
- scoped_refptr<Configurator> config,
- PersistedData* metadata) {
- return std::make_unique<MockUpdateChecker>();
- }
- void CheckForUpdates(
- scoped_refptr<UpdateContext> context,
- const base::flat_map<std::string, std::string>& additional_attributes,
- UpdateCheckCallback update_check_callback) override {
- EXPECT_EQ(1u, context->components_to_check_for_updates.size());
- const std::string id = "jebgalgnebhfojomionfpkfelancnnkf";
- EXPECT_EQ(id, context->components_to_check_for_updates[0]);
- EXPECT_EQ(1u, context->components.count(id));
- ProtocolParser::Result::Manifest::Package package;
- package.name = "jebgalgnebhfojomionfpkfelancnnkf.crx";
- package.hash_sha256 =
- "7ab32f071cd9b5ef8e0d7913be161f532d98b3e9fa284a7cd8059c3409ce0498";
- package.fingerprint = "some-fingerprint";
- ProtocolParser::Result result;
- result.extension_id = id;
- result.status = "ok";
- result.crx_urls.emplace_back("http://localhost/download/");
- result.manifest.version = "1.0";
- result.manifest.browser_min_version = "11.0.1.0";
- result.manifest.run = "UpdaterSetup.exe";
- result.manifest.arguments = "--arg1 --arg2";
- result.manifest.packages.push_back(package);
- ProtocolParser::Results results;
- results.list.push_back(result);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(update_check_callback), results,
- ErrorCategory::kNone, 0, 0));
- }
- };
- class MockCrxDownloader : public CrxDownloader {
- public:
- MockCrxDownloader() : CrxDownloader(nullptr) {}
- private:
- ~MockCrxDownloader() override = default;
- void DoStartDownload(const GURL& url) override {
- DownloadMetrics download_metrics;
- FilePath path;
- Result result;
- if (url.path() == "/download/jebgalgnebhfojomionfpkfelancnnkf.crx") {
- download_metrics.url = url;
- download_metrics.downloader = DownloadMetrics::kNone;
- download_metrics.error = 0;
- download_metrics.downloaded_bytes = 1843;
- download_metrics.total_bytes = 1843;
- download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
- result.error = 0;
- result.response = path;
- } else {
- NOTREACHED();
- }
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadProgress,
- base::Unretained(this),
- download_metrics.downloaded_bytes,
- download_metrics.total_bytes));
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(&MockCrxDownloader::OnDownloadComplete,
- base::Unretained(this), true, result,
- download_metrics));
- }
- };
- class MockPingManager : public MockPingManagerImpl {
- public:
- explicit MockPingManager(scoped_refptr<Configurator> config)
- : MockPingManagerImpl(config) {}
- protected:
- ~MockPingManager() override {
- const auto ping_data = MockPingManagerImpl::ping_data();
- EXPECT_EQ(1u, ping_data.size());
- EXPECT_EQ("jebgalgnebhfojomionfpkfelancnnkf", ping_data[0].id);
- EXPECT_EQ(base::Version("0.0"), ping_data[0].previous_version);
- EXPECT_EQ(base::Version("1.0"), ping_data[0].next_version);
- EXPECT_EQ(ErrorCategory::kService, ping_data[0].error_category);
- EXPECT_EQ(static_cast<int>(ServiceError::CANCELLED),
- ping_data[0].error_code);
- }
- };
- SetMockCrxDownloader<MockCrxDownloader>();
- scoped_refptr<UpdateClient> update_client =
- base::MakeRefCounted<UpdateClientImpl>(
- config(), base::MakeRefCounted<MockPingManager>(config()),
- &MockUpdateChecker::Create);
- base::RepeatingClosure cancel;
- MockObserver observer;
- {
- InSequence seq;
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_CHECKING_FOR_UPDATES,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_FOUND,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1)
- .WillOnce(Invoke([&cancel]() { cancel.Run(); }));
- EXPECT_CALL(observer, OnEvent(Events::COMPONENT_UPDATE_ERROR,
- "jebgalgnebhfojomionfpkfelancnnkf"))
- .Times(1);
- }
- std::vector<CrxUpdateItem> items;
- auto receiver = base::MakeRefCounted<MockCrxStateChangeReceiver>();
- EXPECT_CALL(*receiver, Receive(_))
- .WillRepeatedly(
- [&items](const CrxUpdateItem& item) { items.push_back(item); });
- update_client->AddObserver(&observer);
- cancel = update_client->Install(
- std::string("jebgalgnebhfojomionfpkfelancnnkf"),
- base::BindOnce(&DataCallbackMock::Callback),
- base::BindRepeating(&MockCrxStateChangeReceiver::Receive, receiver),
- base::BindOnce(&CompletionCallbackMock::Callback, quit_closure()));
- RunThreads();
- EXPECT_EQ(3u, items.size());
- EXPECT_EQ(ComponentState::kChecking, items[0].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[0].id.c_str());
- EXPECT_EQ(ComponentState::kCanUpdate, items[1].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[1].id.c_str());
- EXPECT_EQ(ComponentState::kUpdateError, items[2].state);
- EXPECT_STREQ("jebgalgnebhfojomionfpkfelancnnkf", items[2].id.c_str());
- update_client->RemoveObserver(&observer);
- }
- } // namespace update_client
|