1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540154115421543154415451546154715481549155015511552155315541555155615571558155915601561156215631564156515661567156815691570157115721573157415751576157715781579158015811582158315841585158615871588158915901591159215931594159515961597159815991600160116021603160416051606160716081609161016111612161316141615161616171618161916201621162216231624162516261627162816291630163116321633163416351636163716381639164016411642164316441645164616471648164916501651165216531654165516561657165816591660166116621663166416651666166716681669167016711672167316741675167616771678167916801681168216831684168516861687168816891690169116921693169416951696169716981699170017011702170317041705170617071708170917101711171217131714171517161717171817191720172117221723172417251726172717281729173017311732173317341735173617371738173917401741174217431744174517461747174817491750175117521753175417551756175717581759176017611762176317641765176617671768176917701771177217731774177517761777177817791780178117821783178417851786178717881789179017911792179317941795179617971798179918001801180218031804180518061807180818091810181118121813181418151816181718181819182018211822182318241825182618271828182918301831183218331834183518361837183818391840184118421843184418451846184718481849185018511852185318541855185618571858185918601861186218631864186518661867186818691870187118721873187418751876187718781879188018811882188318841885188618871888188918901891189218931894189518961897189818991900190119021903190419051906190719081909191019111912191319141915191619171918191919201921192219231924192519261927192819291930193119321933193419351936193719381939194019411942194319441945194619471948194919501951195219531954195519561957195819591960196119621963196419651966196719681969197019711972197319741975197619771978197919801981198219831984198519861987198819891990199119921993199419951996199719981999200020012002200320042005200620072008200920102011201220132014201520162017201820192020202120222023202420252026202720282029203020312032203320342035203620372038203920402041204220432044204520462047204820492050205120522053205420552056205720582059206020612062206320642065206620672068206920702071207220732074207520762077207820792080208120822083208420852086208720882089209020912092209320942095209620972098209921002101210221032104210521062107210821092110211121122113211421152116211721182119212021212122212321242125212621272128212921302131213221332134213521362137213821392140214121422143214421452146214721482149215021512152215321542155215621572158215921602161216221632164216521662167216821692170217121722173217421752176217721782179218021812182218321842185218621872188218921902191219221932194219521962197219821992200220122022203220422052206220722082209221022112212221322142215221622172218221922202221222222232224222522262227222822292230223122322233223422352236223722382239224022412242224322442245224622472248224922502251225222532254225522562257225822592260226122622263226422652266226722682269227022712272227322742275227622772278227922802281228222832284228522862287228822892290229122922293229422952296229722982299230023012302230323042305230623072308230923102311231223132314231523162317231823192320232123222323232423252326232723282329233023312332233323342335233623372338233923402341234223432344234523462347234823492350235123522353235423552356235723582359236023612362236323642365236623672368236923702371237223732374237523762377237823792380238123822383238423852386238723882389239023912392239323942395239623972398239924002401240224032404240524062407240824092410241124122413241424152416241724182419242024212422242324242425242624272428242924302431243224332434243524362437243824392440244124422443244424452446244724482449245024512452245324542455245624572458245924602461246224632464246524662467246824692470247124722473247424752476247724782479248024812482248324842485248624872488248924902491249224932494249524962497249824992500250125022503250425052506250725082509251025112512251325142515251625172518251925202521252225232524252525262527252825292530253125322533253425352536253725382539254025412542254325442545254625472548254925502551255225532554255525562557255825592560256125622563256425652566256725682569257025712572257325742575257625772578257925802581258225832584258525862587258825892590259125922593259425952596259725982599260026012602260326042605260626072608260926102611261226132614261526162617261826192620262126222623262426252626262726282629263026312632263326342635263626372638263926402641264226432644264526462647264826492650265126522653265426552656265726582659266026612662266326642665266626672668266926702671267226732674267526762677267826792680268126822683268426852686268726882689269026912692269326942695269626972698269927002701270227032704270527062707270827092710271127122713271427152716271727182719272027212722272327242725272627272728272927302731273227332734273527362737273827392740274127422743274427452746274727482749275027512752275327542755275627572758275927602761276227632764276527662767276827692770277127722773277427752776277727782779278027812782278327842785278627872788278927902791279227932794279527962797279827992800280128022803280428052806280728082809281028112812281328142815281628172818281928202821282228232824282528262827282828292830283128322833283428352836283728382839284028412842284328442845284628472848284928502851285228532854285528562857285828592860286128622863286428652866286728682869287028712872287328742875287628772878287928802881288228832884288528862887288828892890289128922893289428952896289728982899290029012902290329042905290629072908290929102911291229132914291529162917291829192920292129222923292429252926292729282929293029312932293329342935293629372938293929402941294229432944294529462947294829492950295129522953295429552956295729582959296029612962296329642965296629672968296929702971297229732974297529762977297829792980298129822983298429852986298729882989299029912992299329942995299629972998299930003001300230033004300530063007300830093010301130123013301430153016301730183019302030213022302330243025302630273028302930303031303230333034303530363037303830393040304130423043304430453046304730483049305030513052305330543055305630573058305930603061306230633064306530663067306830693070307130723073307430753076307730783079308030813082308330843085308630873088308930903091309230933094309530963097309830993100310131023103310431053106310731083109311031113112311331143115311631173118311931203121312231233124312531263127312831293130313131323133313431353136313731383139314031413142314331443145314631473148314931503151315231533154315531563157315831593160316131623163316431653166316731683169317031713172317331743175317631773178317931803181318231833184318531863187318831893190319131923193319431953196319731983199320032013202320332043205320632073208320932103211321232133214321532163217321832193220322132223223322432253226322732283229323032313232323332343235323632373238323932403241324232433244324532463247324832493250325132523253325432553256325732583259326032613262326332643265326632673268326932703271327232733274327532763277327832793280328132823283328432853286328732883289329032913292329332943295329632973298329933003301330233033304330533063307330833093310331133123313331433153316331733183319332033213322332333243325332633273328332933303331333233333334333533363337333833393340334133423343334433453346334733483349335033513352335333543355335633573358335933603361336233633364336533663367336833693370337133723373337433753376337733783379338033813382338333843385338633873388338933903391339233933394339533963397339833993400340134023403340434053406340734083409341034113412341334143415341634173418341934203421342234233424342534263427342834293430343134323433343434353436343734383439344034413442344334443445344634473448344934503451345234533454345534563457345834593460346134623463346434653466346734683469347034713472347334743475347634773478347934803481348234833484348534863487348834893490349134923493349434953496349734983499350035013502350335043505350635073508350935103511351235133514351535163517351835193520352135223523352435253526352735283529353035313532353335343535353635373538353935403541354235433544354535463547354835493550355135523553355435553556355735583559356035613562356335643565356635673568356935703571357235733574357535763577357835793580358135823583358435853586358735883589359035913592359335943595359635973598359936003601360236033604360536063607360836093610361136123613361436153616361736183619362036213622362336243625362636273628362936303631363236333634363536363637363836393640364136423643364436453646364736483649365036513652365336543655365636573658365936603661366236633664366536663667366836693670367136723673367436753676367736783679368036813682368336843685368636873688368936903691369236933694369536963697369836993700370137023703370437053706370737083709371037113712371337143715371637173718371937203721372237233724372537263727372837293730373137323733373437353736373737383739374037413742374337443745374637473748374937503751375237533754375537563757375837593760376137623763376437653766376737683769377037713772377337743775377637773778377937803781378237833784378537863787378837893790379137923793379437953796379737983799380038013802380338043805380638073808380938103811381238133814381538163817381838193820382138223823382438253826382738283829383038313832383338343835383638373838383938403841384238433844384538463847384838493850385138523853385438553856385738583859386038613862386338643865386638673868386938703871387238733874387538763877387838793880388138823883388438853886388738883889389038913892389338943895389638973898389939003901390239033904390539063907390839093910391139123913391439153916391739183919392039213922392339243925392639273928392939303931393239333934393539363937393839393940394139423943394439453946394739483949395039513952395339543955395639573958395939603961396239633964396539663967396839693970397139723973397439753976397739783979398039813982398339843985398639873988398939903991399239933994399539963997399839994000400140024003400440054006400740084009401040114012401340144015401640174018401940204021402240234024402540264027402840294030403140324033403440354036403740384039404040414042404340444045404640474048404940504051405240534054405540564057405840594060406140624063406440654066406740684069407040714072407340744075407640774078407940804081408240834084408540864087408840894090409140924093409440954096409740984099410041014102410341044105410641074108410941104111411241134114411541164117411841194120412141224123412441254126412741284129413041314132413341344135413641374138413941404141414241434144414541464147414841494150415141524153415441554156415741584159416041614162416341644165416641674168416941704171417241734174417541764177417841794180418141824183418441854186418741884189419041914192419341944195419641974198419942004201420242034204420542064207420842094210421142124213421442154216421742184219422042214222422342244225422642274228422942304231423242334234423542364237423842394240424142424243424442454246424742484249425042514252425342544255425642574258425942604261426242634264426542664267426842694270427142724273427442754276427742784279428042814282428342844285428642874288428942904291429242934294429542964297429842994300430143024303430443054306430743084309431043114312431343144315431643174318431943204321432243234324432543264327432843294330433143324333433443354336433743384339434043414342434343444345434643474348434943504351435243534354435543564357435843594360436143624363436443654366436743684369437043714372437343744375437643774378437943804381438243834384438543864387438843894390439143924393439443954396439743984399440044014402440344044405440644074408440944104411441244134414441544164417441844194420442144224423442444254426442744284429443044314432443344344435443644374438443944404441444244434444444544464447444844494450445144524453445444554456445744584459446044614462446344644465446644674468446944704471447244734474447544764477447844794480448144824483448444854486448744884489449044914492449344944495449644974498449945004501450245034504450545064507450845094510451145124513451445154516451745184519452045214522452345244525452645274528452945304531453245334534453545364537453845394540454145424543454445454546454745484549455045514552455345544555455645574558455945604561456245634564456545664567456845694570457145724573457445754576457745784579458045814582458345844585458645874588458945904591459245934594459545964597459845994600460146024603460446054606460746084609461046114612461346144615461646174618461946204621462246234624462546264627462846294630463146324633463446354636463746384639464046414642464346444645464646474648464946504651465246534654465546564657465846594660466146624663466446654666466746684669467046714672467346744675467646774678467946804681468246834684468546864687468846894690469146924693469446954696469746984699470047014702470347044705470647074708470947104711471247134714471547164717471847194720472147224723472447254726472747284729473047314732473347344735473647374738473947404741474247434744474547464747474847494750475147524753475447554756475747584759476047614762476347644765476647674768476947704771477247734774477547764777477847794780478147824783478447854786478747884789479047914792479347944795479647974798479948004801480248034804480548064807480848094810481148124813481448154816481748184819482048214822482348244825482648274828482948304831483248334834483548364837483848394840484148424843484448454846484748484849485048514852485348544855485648574858485948604861486248634864486548664867486848694870487148724873487448754876487748784879488048814882488348844885488648874888488948904891489248934894489548964897489848994900490149024903490449054906490749084909491049114912491349144915491649174918491949204921492249234924492549264927492849294930493149324933493449354936493749384939494049414942494349444945494649474948494949504951495249534954495549564957495849594960496149624963496449654966496749684969497049714972497349744975497649774978497949804981498249834984498549864987498849894990499149924993499449954996499749984999500050015002500350045005500650075008500950105011501250135014501550165017501850195020502150225023502450255026502750285029503050315032503350345035503650375038503950405041504250435044504550465047504850495050505150525053505450555056505750585059506050615062506350645065506650675068506950705071507250735074507550765077507850795080508150825083508450855086508750885089509050915092509350945095509650975098509951005101510251035104510551065107510851095110511151125113511451155116511751185119512051215122512351245125512651275128512951305131513251335134513551365137513851395140514151425143514451455146514751485149515051515152515351545155515651575158515951605161516251635164516551665167516851695170517151725173517451755176517751785179518051815182518351845185518651875188518951905191519251935194519551965197519851995200520152025203520452055206520752085209521052115212521352145215521652175218521952205221522252235224522552265227522852295230523152325233523452355236523752385239524052415242524352445245524652475248524952505251525252535254525552565257525852595260526152625263526452655266526752685269527052715272527352745275527652775278527952805281528252835284528552865287528852895290529152925293529452955296529752985299530053015302530353045305530653075308530953105311531253135314531553165317531853195320532153225323532453255326532753285329533053315332533353345335533653375338533953405341534253435344534553465347534853495350535153525353535453555356535753585359536053615362536353645365536653675368536953705371537253735374537553765377537853795380538153825383538453855386538753885389539053915392539353945395539653975398539954005401540254035404540554065407540854095410541154125413541454155416541754185419542054215422542354245425542654275428542954305431543254335434543554365437543854395440544154425443544454455446544754485449545054515452545354545455545654575458545954605461546254635464546554665467546854695470547154725473547454755476547754785479548054815482548354845485548654875488548954905491549254935494549554965497549854995500550155025503550455055506550755085509551055115512551355145515551655175518551955205521552255235524552555265527552855295530553155325533553455355536553755385539554055415542554355445545554655475548554955505551555255535554555555565557555855595560556155625563556455655566556755685569557055715572557355745575557655775578557955805581558255835584558555865587558855895590559155925593559455955596559755985599560056015602560356045605560656075608560956105611561256135614561556165617561856195620562156225623562456255626562756285629563056315632563356345635563656375638563956405641564256435644564556465647564856495650565156525653565456555656565756585659566056615662566356645665566656675668566956705671567256735674567556765677567856795680568156825683568456855686568756885689569056915692569356945695569656975698569957005701570257035704570557065707570857095710571157125713571457155716571757185719572057215722572357245725572657275728572957305731573257335734573557365737573857395740574157425743574457455746574757485749575057515752575357545755575657575758575957605761576257635764576557665767576857695770577157725773577457755776577757785779578057815782578357845785578657875788578957905791579257935794579557965797579857995800580158025803580458055806580758085809581058115812581358145815581658175818581958205821582258235824582558265827582858295830583158325833583458355836583758385839584058415842584358445845584658475848584958505851585258535854585558565857585858595860586158625863586458655866586758685869587058715872587358745875587658775878587958805881588258835884588558865887588858895890589158925893589458955896589758985899590059015902590359045905590659075908590959105911591259135914591559165917591859195920592159225923592459255926592759285929593059315932593359345935593659375938593959405941594259435944594559465947594859495950595159525953595459555956595759585959596059615962596359645965596659675968596959705971597259735974597559765977597859795980598159825983598459855986598759885989599059915992599359945995599659975998599960006001600260036004600560066007600860096010601160126013601460156016601760186019602060216022602360246025602660276028602960306031603260336034603560366037603860396040604160426043604460456046604760486049605060516052605360546055605660576058605960606061606260636064606560666067606860696070607160726073607460756076607760786079608060816082608360846085608660876088608960906091609260936094609560966097609860996100610161026103610461056106610761086109611061116112611361146115611661176118611961206121612261236124612561266127612861296130613161326133613461356136613761386139614061416142614361446145614661476148614961506151615261536154615561566157615861596160616161626163616461656166616761686169617061716172617361746175617661776178617961806181618261836184618561866187618861896190619161926193619461956196619761986199620062016202620362046205620662076208620962106211621262136214621562166217621862196220622162226223622462256226622762286229623062316232623362346235623662376238623962406241624262436244624562466247624862496250625162526253625462556256625762586259626062616262626362646265626662676268626962706271627262736274627562766277627862796280628162826283628462856286628762886289629062916292629362946295629662976298629963006301630263036304630563066307630863096310631163126313631463156316631763186319632063216322632363246325632663276328632963306331633263336334633563366337633863396340634163426343634463456346634763486349635063516352635363546355635663576358635963606361636263636364636563666367636863696370637163726373637463756376637763786379638063816382638363846385638663876388638963906391639263936394639563966397639863996400640164026403640464056406640764086409641064116412641364146415641664176418641964206421642264236424642564266427642864296430643164326433643464356436643764386439644064416442644364446445644664476448644964506451645264536454645564566457645864596460646164626463646464656466646764686469647064716472647364746475647664776478647964806481648264836484648564866487648864896490649164926493649464956496649764986499650065016502650365046505650665076508650965106511651265136514651565166517651865196520652165226523652465256526652765286529653065316532653365346535653665376538653965406541654265436544654565466547654865496550655165526553655465556556655765586559656065616562656365646565656665676568656965706571657265736574657565766577657865796580658165826583658465856586658765886589659065916592659365946595659665976598659966006601660266036604660566066607660866096610661166126613661466156616661766186619662066216622662366246625662666276628662966306631663266336634663566366637663866396640664166426643664466456646664766486649665066516652665366546655665666576658665966606661666266636664666566666667666866696670667166726673667466756676667766786679668066816682668366846685668666876688668966906691669266936694669566966697669866996700670167026703670467056706670767086709671067116712671367146715671667176718671967206721672267236724672567266727672867296730673167326733673467356736673767386739674067416742674367446745674667476748674967506751675267536754675567566757675867596760676167626763676467656766676767686769677067716772677367746775677667776778677967806781678267836784678567866787678867896790679167926793679467956796679767986799680068016802680368046805680668076808680968106811681268136814681568166817681868196820682168226823682468256826682768286829683068316832683368346835683668376838683968406841684268436844684568466847684868496850685168526853685468556856685768586859686068616862686368646865686668676868686968706871687268736874687568766877687868796880688168826883688468856886688768886889689068916892689368946895689668976898689969006901690269036904690569066907690869096910691169126913691469156916691769186919692069216922692369246925692669276928692969306931693269336934693569366937693869396940694169426943694469456946694769486949695069516952695369546955695669576958695969606961696269636964696569666967696869696970697169726973697469756976697769786979698069816982698369846985698669876988698969906991699269936994699569966997699869997000700170027003700470057006700770087009701070117012701370147015701670177018701970207021702270237024702570267027702870297030703170327033703470357036703770387039704070417042704370447045704670477048704970507051705270537054705570567057705870597060706170627063706470657066706770687069707070717072707370747075707670777078707970807081708270837084708570867087708870897090709170927093709470957096709770987099710071017102710371047105710671077108710971107111711271137114711571167117711871197120712171227123712471257126712771287129713071317132713371347135713671377138713971407141714271437144714571467147714871497150715171527153715471557156715771587159716071617162716371647165716671677168716971707171717271737174717571767177717871797180718171827183718471857186718771887189719071917192719371947195719671977198719972007201720272037204720572067207720872097210721172127213721472157216721772187219722072217222722372247225722672277228722972307231723272337234723572367237723872397240724172427243724472457246724772487249725072517252725372547255725672577258725972607261726272637264726572667267726872697270727172727273727472757276727772787279728072817282728372847285728672877288728972907291729272937294729572967297729872997300730173027303730473057306730773087309731073117312731373147315731673177318731973207321732273237324732573267327732873297330733173327333733473357336733773387339734073417342734373447345734673477348734973507351735273537354735573567357735873597360736173627363736473657366736773687369737073717372737373747375737673777378737973807381738273837384738573867387738873897390739173927393739473957396739773987399740074017402740374047405740674077408740974107411741274137414741574167417741874197420742174227423742474257426742774287429743074317432743374347435743674377438743974407441744274437444744574467447744874497450745174527453745474557456745774587459746074617462746374647465746674677468746974707471747274737474747574767477747874797480748174827483748474857486748774887489749074917492749374947495749674977498749975007501750275037504750575067507750875097510751175127513751475157516751775187519752075217522752375247525752675277528752975307531753275337534753575367537753875397540754175427543754475457546754775487549755075517552755375547555755675577558755975607561756275637564756575667567756875697570757175727573757475757576757775787579758075817582758375847585758675877588758975907591759275937594759575967597759875997600760176027603760476057606760776087609761076117612761376147615761676177618761976207621762276237624762576267627762876297630763176327633763476357636763776387639764076417642764376447645764676477648764976507651765276537654765576567657765876597660766176627663766476657666766776687669767076717672767376747675767676777678767976807681768276837684768576867687768876897690769176927693769476957696769776987699770077017702770377047705770677077708770977107711771277137714771577167717771877197720772177227723772477257726772777287729773077317732773377347735773677377738773977407741774277437744774577467747774877497750775177527753775477557756775777587759776077617762776377647765776677677768776977707771777277737774777577767777777877797780778177827783778477857786778777887789779077917792779377947795779677977798779978007801780278037804780578067807780878097810781178127813781478157816781778187819782078217822782378247825782678277828782978307831783278337834783578367837783878397840784178427843784478457846784778487849785078517852785378547855785678577858785978607861786278637864786578667867786878697870787178727873787478757876787778787879788078817882788378847885788678877888788978907891789278937894789578967897789878997900790179027903790479057906790779087909791079117912791379147915791679177918791979207921792279237924792579267927792879297930793179327933793479357936793779387939794079417942794379447945794679477948794979507951795279537954795579567957795879597960796179627963796479657966796779687969 |
- # -*- coding: utf-8 -*-
- """
- (c) 2015-2017 - Copyright Red Hat Inc
- Authors:
- Pierre-Yves Chibon <pingou@pingoured.fr>
- """
- from __future__ import unicode_literals, absolute_import
- import json
- import unittest
- import shutil
- import sys
- import tempfile
- import time
- import os
- import re
- import pagure_messages
- import pygit2
- import six
- from bs4 import BeautifulSoup
- from datetime import datetime, timedelta
- from fedora_messaging import api, testing
- from mock import ANY, patch, MagicMock
- sys.path.insert(
- 0, os.path.join(os.path.dirname(os.path.abspath(__file__)), "..")
- )
- import pagure.lib.query
- import pagure.lib.tasks
- import tests
- from pagure.lib.repo import PagureRepo
- def _get_commits(output):
- """Returns the commits message in the output. All commits must have
- been made by `Alice Author` or `PY C` to be found.
- """
- commits = []
- save = False
- cnt = 0
- for row in output.split("\n"):
- if row.strip() in ["Alice Author", "Alice Äuthòr", "PY C"]:
- save = True
- if save:
- cnt += 1
- if cnt == 7:
- commits.append(row.strip())
- save = False
- cnt = 0
- return commits
- MERGED_PATTERN = (
- re.escape('<span class="text-info font-weight-bold">Merged</span> ')
- + "(just now|seconds ago)\n"
- + re.escape(
- " </span>\n by\n"
- ' <span title="PY C (pingou)">pingou.</span>\n'
- )
- )
- def set_up_git_repo(
- session,
- path,
- new_project=None,
- branch_from="feature",
- mtype="FF",
- prid=1,
- name_from="test",
- ):
- """Set up the git repo and create the corresponding PullRequest
- object.
- """
- # Create a git repo to play with
- gitrepo = os.path.join(path, "repos", "%s.git" % name_from)
- repo = pygit2.init_repository(gitrepo, bare=True)
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- repopath = os.path.join(newpath, "test")
- clone_repo = pygit2.clone_repository(gitrepo, repopath)
- # Create a file in that git repo
- with open(os.path.join(repopath, "sources"), "w") as stream:
- stream.write("foo\n bar")
- clone_repo.index.add("sources")
- clone_repo.index.write()
- try:
- com = repo.revparse_single("HEAD")
- prev_commit = [com.oid.hex]
- except:
- prev_commit = []
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- clone_repo.create_commit(
- "refs/heads/master", # the name of the reference to update
- author,
- committer,
- "Add sources file for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- prev_commit,
- )
- refname = "refs/heads/master:refs/heads/master"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- first_commit = repo.revparse_single("HEAD")
- def compatible_signature(name, email):
- if six.PY2:
- name = name.encode("utf-8")
- email = email.encode("utf-8")
- return pygit2.Signature(name, email)
- if mtype == "merge":
- with open(os.path.join(repopath, ".gitignore"), "w") as stream:
- stream.write("*~")
- clone_repo.index.add(".gitignore")
- clone_repo.index.write()
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = compatible_signature("Alice Äuthòr", "alice@äuthòrs.tld")
- comitter = compatible_signature(
- "Cecil Cõmmîttër", "cecil@cõmmîttërs.tld"
- )
- clone_repo.create_commit(
- "refs/heads/master",
- author,
- committer,
- "Add .gitignore file for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [first_commit.oid.hex],
- )
- refname = "refs/heads/master:refs/heads/master"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- if mtype == "conflicts":
- with open(os.path.join(repopath, "sources"), "w") as stream:
- stream.write("foo\n bar\nbaz")
- clone_repo.index.add("sources")
- clone_repo.index.write()
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- clone_repo.create_commit(
- "refs/heads/master",
- author,
- committer,
- "Add sources conflicting",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [first_commit.oid.hex],
- )
- refname = "refs/heads/master:refs/heads/master"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- # Set the second repo
- new_gitrepo = repopath
- if new_project:
- # Create a new git repo to play with
- new_gitrepo = os.path.join(newpath, new_project.fullname)
- if not os.path.exists(new_gitrepo):
- os.makedirs(new_gitrepo)
- new_repo = pygit2.clone_repository(gitrepo, new_gitrepo)
- repo = pygit2.Repository(new_gitrepo)
- if mtype != "nochanges":
- # Edit the sources file again
- with open(os.path.join(new_gitrepo, "sources"), "w") as stream:
- stream.write("foo\n bar\nbaz\n boose")
- repo.index.add("sources")
- repo.index.write()
- # Commits the files added
- tree = repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- repo.create_commit(
- "refs/heads/%s" % branch_from,
- author,
- committer,
- "A commit on branch %s\n\nMore information" % branch_from,
- tree,
- [first_commit.oid.hex],
- )
- refname = "refs/heads/%s" % (branch_from)
- ori_remote = repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- # Create a PR for these changes
- project = pagure.lib.query.get_authorized_project(session, "test")
- req = pagure.lib.query.new_pull_request(
- session=session,
- repo_from=project,
- branch_from=branch_from,
- repo_to=project,
- branch_to="master",
- title="PR from the %s branch" % branch_from,
- user="pingou",
- )
- session.commit()
- assert req.id == prid
- assert req.title == "PR from the %s branch" % branch_from
- shutil.rmtree(newpath)
- class PagureFlaskForktests(tests.Modeltests):
- """Tests for flask fork controller of pagure"""
- def test_request_pull_reference(self):
- """Test if there is a reference created for a new PR."""
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- # View the pull-request
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- gitrepo = os.path.join(self.path, "repos", "test.git")
- repo = pygit2.Repository(gitrepo)
- self.assertEqual(
- list(repo.listall_references()),
- ["refs/heads/feature", "refs/heads/master", "refs/pull/1/head"],
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull(self, send_email):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- # Non-existant project
- output = self.app.get("/foobar/pull-request/1")
- self.assertEqual(output.status_code, 404)
- # Project has no PR
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 404)
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- # View the pull-request
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # self.assertIn(
- #'<h3><span class="label label-default">PR#1</span>\n'
- #' PR from the feature branch\n</h3>',
- # output_text)
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- # Test if the `open changed file icon` is displayed.
- self.assertIn(
- 'class="open_changed_file_icon_wrap"><span '
- 'class="fa fa-file-code-o fa-fw" '
- 'alt="Open changed file" title="Open changed file"></span>'
- "</a>",
- output_text,
- )
- self.assertIn(
- '<span class="btn btn-success btn-sm font-weight-bold disabled'
- ' opacity-100">+3</span>',
- output_text,
- )
- self.assertIn(
- '<span class="btn btn-danger btn-sm font-weight-bold disabled '
- 'opacity-100">-1</span>',
- output_text,
- )
- # Test if hunk headline is rendered without line numbers
- self.assertIn(
- '<td class="cell1"></td><td class="prc border-right"></td>\n<td '
- 'class="cell2 stretch-table-column"> <pre class='
- '"text-muted"><code>@@ -1,2 +1,4 @@',
- output_text,
- )
- # Tests if line number 1 is displayed
- self.assertNotIn(
- '<td class="cell1"><a id="_1__1" href="#_1__1" data-line-number="1" data-file-number="1"></a></td>',
- output_text,
- )
- # Test if line number 2 is displayed
- self.assertIn(
- '<td class="cell1"><a id="_1__2" href="#_1__2" data-line-number="2" data-file-number="1"></a></td>',
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_delete_branch_button_no_auth(self, send_email):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- # View the pull-request
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- # Un-authenticated user cannot see this checkbox
- self.assertNotIn(
- '<input id="delete_branch" name="delete_branch" type="checkbox" '
- 'value="y"> <label for="delete_branch">Delete branch after '
- "merging</label>",
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_delete_branch_button(self, send_email):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- # View the pull-request
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- self.assertIn(
- '<input id="delete_branch" name="delete_branch" type="checkbox" '
- 'value="y"> <label for="delete_branch">Delete branch after '
- "merging</label>",
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_delete_branch_button_no_project_from(
- self, send_email
- ):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- project.requests[0].project_from = None
- self.session.add(project.requests[0])
- self.session.commit()
- # View the pull-request
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- self.assertIn(
- '<input id="delete_branch" name="delete_branch" type="checkbox" '
- 'value="y"> <label for="delete_branch">Delete branch after '
- "merging</label>",
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_delete_branch_button_no_project_from_no_acl(
- self, send_email
- ):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- project.requests[0].project_from = None
- self.session.add(project.requests[0])
- self.session.commit()
- # View the pull-request
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- self.assertNotIn(
- '<input id="delete_branch" name="delete_branch" type="checkbox" '
- 'value="y"> <label for="delete_branch">Delete branch after '
- "merging</label>",
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_task_update_request_pull(self, send_email):
- """Test the task update_pull_request endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- self.session = pagure.lib.query.create_session(self.dbpath)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- request = project.requests[0]
- self.assertEqual(len(request.comments), 0)
- start_commit = request.commit_start
- stop_commit = request.commit_stop
- # View the pull-request
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- # Add a new commit on the repo from
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "test.git")
- repopath = os.path.join(newpath, "test")
- clone_repo = pygit2.clone_repository(
- gitrepo, repopath, checkout_branch="feature"
- )
- def compatible_signature(name, email):
- if six.PY2:
- name = name.encode("utf-8")
- email = email.encode("utf-8")
- return pygit2.Signature(name, email)
- with open(os.path.join(repopath, ".gitignore"), "w") as stream:
- stream.write("*~")
- clone_repo.index.add(".gitignore")
- clone_repo.index.write()
- com = clone_repo.revparse_single("HEAD")
- prev_commit = [com.oid.hex]
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = compatible_signature("Alice Äuthòr", "alice@äuthòrs.tld")
- comitter = compatible_signature(
- "Cecil Cõmmîttër", "cecil@cõmmîttërs.tld"
- )
- clone_repo.create_commit(
- "refs/heads/feature",
- author,
- comitter,
- "Add .gitignore file for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- prev_commit,
- )
- refname = "refs/heads/feature:refs/heads/feature"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- shutil.rmtree(newpath)
- pagure.lib.tasks.update_pull_request(request.uid)
- self.session = pagure.lib.query.create_session(self.dbpath)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 1)
- request = project.requests[0]
- self.assertEqual(len(request.comments), 1)
- self.assertIsNotNone(request.commit_start)
- self.assertIsNotNone(request.commit_stop)
- self.assertNotEqual(start_commit, request.commit_start)
- self.assertNotEqual(stop_commit, request.commit_stop)
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_ci_dropdown(self, send_email):
- """Test presence of the "Rerun CI" dropdown with various settings."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # old-style TRIGGER_CI list - test backwards compatibility
- with patch.dict(
- "pagure.config.config",
- {"TRIGGER_CI": ["old-style-trigger-ci"]},
- ):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertNotIn("Rerun CI", output_text)
- # new-style TRIGGER_CI, but no button to show
- with patch.dict(
- "pagure.config.config", {"TRIGGER_CI": {"no-button": None}}
- ):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertNotIn("Rerun CI", output_text)
- trigger_ci = {
- "foobar-ci": {
- "name": "foobar-ci-name",
- "description": "barfoo",
- },
- "spam-ci": {
- "name": "spam-ci-name",
- "description": "with beans and eggs",
- },
- "no-button-for-me-ci": None,
- }
- # new-style TRIGGER_CI, several buttons to show
- with patch.dict(
- "pagure.config.config", {"TRIGGER_CI": trigger_ci}
- ):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn("Rerun CI", output_text)
- self.assertIn("foobar-ci-name", output_text)
- self.assertIn("spam-ci-name", output_text)
- self.assertNotIn("no-button-for-me-ci", output_text)
- trigger_ci = {
- "foobar-ci": {
- "name": "foobar-ci-name",
- "description": "barfoo",
- "requires_project_hook_attr": (
- "ci_hook",
- "active_pr",
- True,
- ),
- }
- }
- # new-style TRIGGER_CI with requires_project_hook_attr that is
- # not fulfilled by the project
- with patch.dict(
- "pagure.config.config", {"TRIGGER_CI": trigger_ci}
- ):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertNotIn("Rerun CI", output_text)
- # now activate the hook and try again
- data = {
- "active_pr": "y",
- "ci_url": "https://jenkins.fedoraproject.org",
- "ci_job": "ci_job",
- "ci_type": "jenkins",
- "csrf_token": self.get_csrf(),
- }
- output = self.app.post(
- "/test/settings/Pagure CI", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- with patch.dict(
- "pagure.config.config", {"TRIGGER_CI": trigger_ci}
- ):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn("Rerun CI", output_text)
- self.assertIn("foobar-ci-name", output_text)
- # shouldn't show up if user is not logged in
- with patch.dict("pagure.config.config", {"TRIGGER_CI": trigger_ci}):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertNotIn("Rerun CI", output_text)
- @patch("pagure.lib.notify.send_email")
- @patch.dict(
- "pagure.config.config",
- {"TRIGGER_CI": {"CI1": {"name": "CI1", "description": "CI1!"}}},
- )
- def test_request_pull_ci_rerun(self, send_email):
- """Test rerunning CI using button from the "Rerun CI" dropdown."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- user = tests.FakeUser()
- user.username = "pingou"
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- request = project.requests[0]
- with tests.user_set(self.app.application, user):
- # no csrf token
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output = self.app.post(
- "/test/pull-request/1/trigger-ci", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- self.assertIn("Invalid input", output.get_data(as_text=True))
- # no such PR
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output = self.app.post(
- "/test/pull-request/2/trigger-ci", follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # wrong comment
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token, "comment": "this doesnt exist"}
- output = self.app.post(
- "/test/pull-request/1/trigger-ci",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- self.assertIn("Invalid input", output.get_data(as_text=True))
- # everything ok
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token, "comment": "CI1"}
- output = self.app.post(
- "/test/pull-request/1/trigger-ci",
- data=data,
- follow_redirects=True,
- )
- output_text = output.get_data(as_text=True)
- self.assertEqual(output.status_code, 200)
- self.assertIn("<p>CI1</p>", output_text)
- comment = request.comments[0]
- self.assertTrue(comment.notification)
- self.assertEqual(comment.comment, "CI1")
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_FF(self, send_email):
- """Test the merge_request_pull endpoint with a FF PR."""
- send_email.return_value = True
- self.test_request_pull()
- user = tests.FakeUser()
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- # No CSRF
- output = self.app.post(
- "/test/pull-request/1/merge", data={}, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- # self.assertIn(
- #'<h3><span class="label label-default">PR#1</span>\n'
- #' PR from the feature branch\n</h3>',
- # output_text)
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- # Wrong project
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/foobar/pull-request/100/merge",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Wrong project
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Wrong request id
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/test/pull-request/100/merge",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Project w/o pull-request
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- # Pull-request disabled
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # Project w pull-request but only assignee can merge
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings["pull_requests"] = True
- settings["Only_assignee_can_merge_pull-request"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn(
- "This request must be " "assigned to be merged", output_text
- )
- # PR assigned but not to this user
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- req = repo.requests[0]
- req.assignee_id = 2
- self.session.add(req)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn(
- "Only the assignee can merge this request", output_text
- )
- # Project w/ minimal PR score
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings["Only_assignee_can_merge_pull-request"] = False
- settings["Minimum_score_to_merge_pull-request"] = 2
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn(
- "This request does not "
- "have the minimum review score necessary to be merged",
- output_text,
- )
- # Merge
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings["Minimum_score_to_merge_pull-request"] = -1
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output = self.app.get("/test/commits")
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Commits - test - Pagure</title>", output_text
- )
- self.assertIn("A commit on branch feature", output_text)
- self.assertNotIn(
- "Merge #1 `PR from the feature branch`", output_text
- )
- # Check if the closing notification was added
- output = self.app.get("/test/pull-request/1")
- self.assertIsNotNone(
- re.search(MERGED_PATTERN, output.get_data(as_text=True))
- )
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_merge(self, send_email):
- """Test the merge_request_pull endpoint with a merge PR."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # Merge
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - Pagure</title>",
- output.get_data(as_text=True),
- )
- # Check if the closing notification was added
- output = self.app.get("/test/pull-request/1")
- self.assertIsNotNone(
- re.search(MERGED_PATTERN, output.get_data(as_text=True))
- )
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_merge_with_comment(self, send_email):
- """Test the merge_request_pull endpoint with a merge PR."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- self.session = pagure.lib.query.create_session(self.dbpath)
- request = pagure.lib.query.search_pull_requests(
- self.session, project_id=1, requestid=1
- )
- self.assertEqual(len(request.comments), 0)
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {
- "csrf_token": csrf_token,
- "comment": "Thanks for the review and the suggestions!",
- }
- # Merge
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - Pagure</title>",
- output.get_data(as_text=True),
- )
- # Check if the closing notification was added
- output = self.app.get("/test/pull-request/1")
- output_text = output.get_data(as_text=True)
- self.assertIsNotNone(re.search(MERGED_PATTERN, output_text))
- self.assertIn(
- "Thanks for the review and the suggestions!", output_text
- )
- self.session = pagure.lib.query.create_session(self.dbpath)
- request = pagure.lib.query.search_pull_requests(
- self.session, project_id=1, requestid=1
- )
- self.assertEqual(len(request.comments), 2)
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_merge_with_delete_branch(self, send_email):
- """Test the merge_request_pull endpoint with a merge PR and delete source branch."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature-branch",
- mtype="merge",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- data = {
- "csrf_token": self.get_csrf(output=output),
- "delete_branch": True,
- }
- # Merge
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature-branch branch - test\n - Pagure</title>",
- output_text,
- )
- # Check the branch is not mentioned
- self.assertNotIn(
- '<a class="" href="/test/branch/feature-branch"', output_text
- )
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_conflicts(self, send_email):
- """Test the merge_request_pull endpoint with a conflicting PR."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="conflicts",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # Merge conflicts
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("Merge conflicts!", output_text)
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_conflicts_with_delete_branch(self, send_email):
- """Test the merge_request_pull endpoint with a conflicting PR and request deletion of branch."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature-branch",
- mtype="conflicts",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- data = {
- "csrf_token": self.get_csrf(output=output),
- "delete_branch": True,
- }
- # Merge conflicts
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n'
- ' <span class="text-success font-weight-bold">#1</span>\n'
- ' <span class="font-weight-bold">\n'
- " PR from the feature-branch branch\n",
- output_text,
- )
- self.assertIn("Merge conflicts!", output_text)
- # Check the branch still exists
- output = self.app.get("/test/branches")
- self.assertIn("feature-branch", output.get_data(as_text=True))
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_nochange(self, send_email):
- """Test the merge_request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="master",
- mtype="nochanges",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # Nothing to merge
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "Nothing to do, changes were already merged", output_text
- )
- # Check if the closing notification was added
- output = self.app.get("/test/pull-request/1")
- output_text = output.get_data(as_text=True)
- self.assertIsNotNone(re.search(MERGED_PATTERN, output_text))
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_close(self, send_email):
- """Test the request_pull endpoint with a closed PR."""
- send_email.return_value = True
- self.test_merge_request_pull_FF()
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIsNotNone(re.search(MERGED_PATTERN, output_text))
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_disabled(self, send_email):
- """Test the request_pull endpoint with PR disabled."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- @patch("pagure.lib.git.update_pull_ref")
- def test_request_pull_empty_repo(self, send_email, update_pull_ref):
- """Test the request_pull endpoint against an empty repo."""
- # Mock update_pull_ref or the repo won't be empty anymore
- # (the PR will have been pushed to refs/pull)
- send_email.return_value = True
- tests.create_projects(self.session)
- item = pagure.lib.model.Project(
- user_id=2, # foo
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(item)
- self.session.commit()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "foo"), bare=True
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- self.assertFalse(os.path.exists(gitrepo))
- os.makedirs(gitrepo)
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- # Edit the sources file again
- with open(os.path.join(newpath, "sources"), "w") as stream:
- stream.write("foo\n bar\nbaz\n boose")
- new_repo.index.add("sources")
- new_repo.index.write()
- # Commits the files added
- tree = new_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- new_repo.create_commit(
- "refs/heads/feature",
- author,
- committer,
- "A commit on branch feature",
- tree,
- [],
- )
- refname = "refs/heads/feature:refs/heads/feature"
- ori_remote = new_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- # Create a PR for these changes
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_from=item,
- branch_from="feature",
- repo_to=project,
- branch_to="master",
- title="PR from the feature branch",
- user="pingou",
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "PR from the feature branch")
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertTrue(output_text.count('<span class="commitdate"'), 1)
- self.assertTrue(update_pull_ref.called)
- shutil.rmtree(newpath)
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_empty_fork(self, send_email):
- """Test the request_pull endpoint from an empty fork."""
- send_email.return_value = True
- tests.create_projects(self.session)
- item = pagure.lib.model.Project(
- user_id=2, # foo
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(item)
- self.session.commit()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "foo"), bare=True
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- self.assertFalse(os.path.exists(gitrepo))
- os.makedirs(gitrepo)
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- # Create a PR for these "changes" (there are none, both repos are
- # empty)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_from=item,
- branch_from="feature",
- repo_to=project,
- branch_to="master",
- title="PR from the feature branch",
- user="pingou",
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "PR from the feature branch")
- output = self.app.get("/test/pull-request/1", follow_redirects=True)
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- "Fork is empty, there are no "
- "commits to create a pull request with",
- output_text,
- )
- shutil.rmtree(newpath)
- @patch("pagure.lib.notify.send_email")
- def test_request_pulls_order(self, send_email):
- """Test the request_pulls
- i.e Make sure that the results are displayed
- in the order required by the user"""
- send_email.return_value = True
- # Initially no project
- output = self.app.get("/test/pull-requests")
- self.assertEqual(output.status_code, 404)
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- item = pagure.lib.model.Project(
- user_id=2,
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(item)
- self.session.commit()
- # create PR's to play with
- # PR-1
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- repo_from=item,
- branch_from="feature",
- branch_to="master",
- title="PR from the feature branch",
- user="pingou",
- status="Open",
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "PR from the feature branch")
- # PR-2
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- branch_to="master",
- branch_from="feature",
- repo_from=item,
- title="test PR",
- user="pingou",
- status="Open",
- )
- self.session.commit()
- self.assertEqual(req.title, "test PR")
- # PR-3
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- branch_from="feature",
- branch_to="master",
- repo_from=item,
- title="test Invalid PR",
- user="pingou",
- status="Closed",
- )
- self.session.commit()
- self.assertEqual(req.title, "test Invalid PR")
- # PR-4
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- branch_from="feature",
- title="test PR for sort",
- repo_from=item,
- user="pingou",
- branch_to="master",
- status="Open",
- )
- self.session.commit()
- self.assertEqual(req.title, "test PR for sort")
- # sort by last_updated
- output = self.app.get("/test/pull-requests?order_key=last_updated")
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(output.status_code, 200)
- # Make sure that issue four is first since it was modified last
- self.assertIn('href="/test/pull-request/4"', tr_elements[0])
- self.assertIn('href="/test/pull-request/2"', tr_elements[1])
- self.assertIn('href="/test/pull-request/1"', tr_elements[2])
- pr_one = pagure.lib.query.search_pull_requests(
- self.session, project_id=1, requestid=1
- )
- pr_one.updated_on = datetime.utcnow() + timedelta(seconds=2)
- self.session.add(pr_one)
- self.session.commit()
- # sort by last_updated
- output = self.app.get("/test/pull-requests?order_key=last_updated")
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(output.status_code, 200)
- # Make sure that PR four is first since it was modified last
- self.assertIn('href="/test/pull-request/1"', tr_elements[0])
- # Make sure that PR two is second since it was modified second
- self.assertIn('href="/test/pull-request/4"', tr_elements[1])
- # Make sure that PR one is last since it was modified first
- self.assertIn('href="/test/pull-request/2"', tr_elements[2])
- # Now query so that the results are ascending
- output = self.app.get(
- "/test/pull-requests?" "order_key=last_updated&order=asc"
- )
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertIn('href="/test/pull-request/2"', tr_elements[0])
- self.assertIn('href="/test/pull-request/4"', tr_elements[1])
- self.assertIn('href="/test/pull-request/1"', tr_elements[2])
- # check that search_pattern argument works
- output = self.app.get("/test/pull-requests?search_pattern=feature")
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertIn('href="/test/pull-request/1"', tr_elements[0])
- self.assertEqual(len(tr_elements), 1)
- output = self.app.get("/test/pull-requests?search_pattern=PR")
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertIn('href="/test/pull-request/4"', tr_elements[0])
- self.assertIn('href="/test/pull-request/2"', tr_elements[1])
- self.assertIn('href="/test/pull-request/1"', tr_elements[2])
- self.assertEqual(len(tr_elements), 3)
- output = self.app.get("/test/pull-requests?search_pattern=*PR")
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(len(tr_elements), 1)
- self.assertIn('href="/test/pull-request/2"', tr_elements[0])
- @patch("pagure.lib.notify.send_email")
- def test_request_pulls(self, send_email):
- """Test the request_pulls endpoint."""
- send_email.return_value = True
- # No such project
- output = self.app.get("/test/pull-requests")
- self.assertEqual(output.status_code, 404)
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- output = self.app.get("/test/pull-requests")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 0 Open PRs\n',
- output_text,
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- output = self.app.get("/test/pull-requests")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 1 Open PRs\n',
- output_text,
- )
- output = self.app.get("/test/pull-requests?status=1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 1 Open PRs\n',
- output_text,
- )
- output = self.app.get("/test/pull-requests?status=true")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 1 Open PRs\n',
- output_text,
- )
- output = self.app.get("/test/pull-requests?status=Merged")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 0 Merged PRs\n',
- output_text,
- )
- output = self.app.get("/test/pull-requests?status=0")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 0 Merged PRs\n',
- output_text,
- )
- output = self.app.get("/test/pull-requests?status=Closed")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-arrow-circle-down"></span> 0 Cancelled PRs\n',
- output_text,
- )
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.get("/test/pull-requests")
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- def test_request_pulls_filters_tags(self, send_email):
- """Test the requests_pull
- i.e Make sure that the results are filtered properly"""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- # Create some tags to play with
- pagure.lib.query.new_tag(
- self.session, "tag-1", "tag-1 descripcion", "#ff0000", repo.id
- )
- pagure.lib.query.new_tag(
- self.session, "tag-2", "tag-2 description", "#00ff00", repo.id
- )
- pagure.lib.query.new_tag(
- self.session, "tag-3", "tag-3 description", "#0000ff", repo.id
- )
- fork = pagure.lib.model.Project(
- user_id=2,
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(fork)
- self.session.commit()
- # Create PR's to play with
- # PR-1, tags: tag-1, tag-3
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- repo_from=fork,
- branch_from="feature",
- branch_to="master",
- title="First PR",
- user="pingou",
- status="Open",
- )
- pagure.lib.query.update_tags(
- self.session, obj=req, tags=["tag-1", "tag-3"], username="pingou"
- )
- self.session.commit()
- # PR-2, tags: tag-2, tag-3
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- repo_from=fork,
- branch_from="feature",
- branch_to="master",
- title="Second PR",
- user="pingou",
- status="Open",
- )
- pagure.lib.query.update_tags(
- self.session, obj=req, tags=["tag-2", "tag-3"], username="pingou"
- )
- self.session.commit()
- # PR-3 closed, tags: tag-1, tag-3
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- repo_from=fork,
- branch_from="feature",
- branch_to="master",
- title="Third PR",
- user="pingou",
- status="Closed",
- )
- pagure.lib.query.update_tags(
- self.session, obj=req, tags=["tag-1", "tag-3"], username="pingou"
- )
- self.session.commit()
- # PR-4 closed, tags: tag-1, tag-2
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_to=repo,
- repo_from=fork,
- branch_from="feature",
- branch_to="master",
- title="Fourth PR",
- user="pingou",
- status="Closed",
- )
- pagure.lib.query.update_tags(
- self.session, obj=req, tags=["tag-1", "tag-2"], username="pingou"
- )
- self.session.commit()
- # filter by 'tag-1'
- output = self.app.get("/test/pull-requests?tags=tag-1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(1, len(tr_elements))
- self.assertIn('href="/test/pull-request/1', tr_elements[0])
- # filter by '!tag-1'
- output = self.app.get("/test/pull-requests?tags=!tag-1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(1, len(tr_elements))
- self.assertIn('href="/test/pull-request/2', tr_elements[0])
- # filter by 'tag-2' and 'tag-3'
- output = self.app.get("/test/pull-requests?tags=tag2&tags=tag-3")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(2, len(tr_elements))
- self.assertIn('href="/test/pull-request/2', tr_elements[0])
- self.assertIn('href="/test/pull-request/1', tr_elements[1])
- # filter by '!tag-3'
- output = self.app.get("/test/pull-requests?tags=!tag-3")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(0, len(tr_elements))
- # filter by tag-2 on Closed prs
- output = self.app.get("/test/pull-requests?status=Closed&tags=tag-2")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(1, len(tr_elements))
- self.assertIn('href="/test/pull-request/4', tr_elements[0])
- # filter by !tag-2 on Closed prs
- output = self.app.get("/test/pull-requests?status=Closed&tags=!tag-2")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(1, len(tr_elements))
- self.assertIn('href="/test/pull-request/3', tr_elements[0])
- # filter by tag-2 on all the prs
- output = self.app.get("/test/pull-requests?status=all&tags=tag-2")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- tr_elements = re.findall(
- '<div class="request-row list-group-item list-group-item-action ">(.*?)</div><!--end request-row-->',
- output_text,
- re.M | re.S,
- )
- self.assertEqual(2, len(tr_elements))
- self.assertIn('href="/test/pull-request/4', tr_elements[0])
- self.assertIn('href="/test/pull-request/2', tr_elements[1])
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_patch(self, send_email):
- """Test the request_pull_patch endpoint."""
- send_email.return_value = True
- output = self.app.get("/test/pull-request/1.patch")
- self.assertEqual(output.status_code, 404)
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- output = self.app.get("/test/pull-request/100.patch")
- self.assertEqual(output.status_code, 404)
- output = self.app.get("/test/pull-request/1.patch")
- self.assertEqual(output.status_code, 200)
- npatch = []
- for row in output.get_data(as_text=True).split("\n"):
- if row.startswith("Date:"):
- continue
- if row.startswith("From "):
- row = row.split(" ", 2)[2]
- npatch.append(row)
- exp = r"""Mon Sep 17 00:00:00 2001
- From: Alice Author <alice@authors.tld>
- Subject: A commit on branch feature
- More information
- ---
- diff --git a/.gitignore b/.gitignore
- new file mode 100644
- index 0000000..e4e5f6c
- --- /dev/null
- +++ b/.gitignore
- @@ -0,0 +1 @@
- +*~
- \ No newline at end of file
- diff --git a/sources b/sources
- index 9f44358..2a552bb 100644
- --- a/sources
- +++ b/sources
- @@ -1,2 +1,4 @@
- foo
- - bar
- \ No newline at end of file
- + bar
- +baz
- + boose
- \ No newline at end of file
- """
- patch = "\n".join(npatch)
- # print patch
- self.assertEqual(patch, exp)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.get("/test/pull-request/1.patch")
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_diff(self, send_email):
- """Test the request_pull_patch endpoint."""
- send_email.return_value = True
- output = self.app.get("/test/pull-request/1.diff")
- self.assertEqual(output.status_code, 404)
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- output = self.app.get("/test/pull-request/100.diff")
- self.assertEqual(output.status_code, 404)
- output = self.app.get("/test/pull-request/1.diff")
- self.assertEqual(output.status_code, 200)
- exp = r"""diff --git a/.gitignore b/.gitignore
- new file mode 100644
- index 0000000..e4e5f6c
- --- /dev/null
- +++ b/.gitignore
- @@ -0,0 +1 @@
- +*~
- \ No newline at end of file
- diff --git a/sources b/sources
- index 9f44358..2a552bb 100644
- --- a/sources
- +++ b/sources
- @@ -1,2 +1,4 @@
- foo
- - bar
- \ No newline at end of file
- + bar
- +baz
- + boose
- \ No newline at end of file
- """
- self.assertEqual(output.get_data(as_text=True), exp)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.get("/test/pull-request/1.diff")
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_patch_close(self, send_email):
- """Test the request_pull_patch endpoint with a closed PR."""
- send_email.return_value = True
- self.test_merge_request_pull_FF()
- output = self.app.get("/test/pull-request/1.patch")
- self.assertEqual(output.status_code, 200)
- npatch = []
- for row in output.get_data(as_text=True).split("\n"):
- if row.startswith("Date:"):
- continue
- if row.startswith("From "):
- row = row.split(" ", 2)[2]
- npatch.append(row)
- exp = r"""Mon Sep 17 00:00:00 2001
- From: Alice Author <alice@authors.tld>
- Subject: A commit on branch feature
- More information
- ---
- diff --git a/sources b/sources
- index 9f44358..2a552bb 100644
- --- a/sources
- +++ b/sources
- @@ -1,2 +1,4 @@
- foo
- - bar
- \ No newline at end of file
- + bar
- +baz
- + boose
- \ No newline at end of file
- """
- patch = "\n".join(npatch)
- # print patch
- self.assertEqual(patch, exp)
- @patch("pagure.lib.notify.send_email")
- @patch("pagure.lib.git.update_pull_ref")
- def test_request_pull_patch_empty_repo(self, send_email, update_pull_ref):
- """Test the request_pull_patch endpoint against an empty repo."""
- # Mock update_pull_ref or the repo won't be empty anymore
- # (the PR will have been pushed to refs/pull)
- send_email.return_value = True
- tests.create_projects(self.session)
- item = pagure.lib.model.Project(
- user_id=2, # foo
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(item)
- self.session.commit()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "foo"), bare=True
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- self.assertFalse(os.path.exists(gitrepo))
- os.makedirs(gitrepo)
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- # Edit the sources file again
- with open(os.path.join(newpath, "sources"), "w") as stream:
- stream.write("foo\n bar\nbaz\n boose")
- new_repo.index.add("sources")
- new_repo.index.write()
- # Commits the files added
- tree = new_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- new_repo.create_commit(
- "refs/heads/feature",
- author,
- committer,
- "A commit on branch feature",
- tree,
- [],
- )
- refname = "refs/heads/feature:refs/heads/feature"
- ori_remote = new_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- # Create a PR for these "changes" (there are none, both repos are
- # empty)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_from=item,
- branch_from="feature",
- repo_to=project,
- branch_to="master",
- title="PR from the feature branch",
- user="pingou",
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "PR from the feature branch")
- output = self.app.get(
- "/test/pull-request/1.patch", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- npatch = []
- for row in output.get_data(as_text=True).split("\n"):
- if row.startswith("Date:"):
- continue
- if row.startswith("From "):
- row = row.split(" ", 2)[2]
- npatch.append(row)
- exp = r"""Mon Sep 17 00:00:00 2001
- From: Alice Author <alice@authors.tld>
- Subject: A commit on branch feature
- ---
- diff --git a/sources b/sources
- new file mode 100644
- index 0000000..2a552bb
- --- /dev/null
- +++ b/sources
- @@ -0,0 +1,4 @@
- +foo
- + bar
- +baz
- + boose
- \ No newline at end of file
- """
- patch = "\n".join(npatch)
- # print patch
- self.assertEqual(patch, exp)
- shutil.rmtree(newpath)
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_patch_empty_fork(self, send_email):
- """Test the request_pull_patch endpoint from an empty fork."""
- send_email.return_value = True
- tests.create_projects(self.session)
- item = pagure.lib.model.Project(
- user_id=2, # foo
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=1,
- )
- self.session.add(item)
- self.session.commit()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "foo"), bare=True
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- self.assertFalse(os.path.exists(gitrepo))
- os.makedirs(gitrepo)
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- # Create a PR for these "changes" (there are none, both repos are
- # empty)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_from=item,
- branch_from="feature",
- repo_to=project,
- branch_to="master",
- title="PR from the feature branch",
- user="pingou",
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "PR from the feature branch")
- output = self.app.get(
- "/test/pull-request/1.patch", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn("<title>Overview - test - Pagure</title>", output_text)
- self.assertIn(
- "Fork is empty, there are no "
- "commits to create a pull request with",
- output_text,
- )
- shutil.rmtree(newpath)
- @patch("pagure.lib.notify.send_email")
- def test_close_request_pull(self, send_email):
- """Test the close_request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- user = tests.FakeUser()
- with tests.user_set(self.app.application, user):
- output = self.app.post("/test/pull-request/close/1")
- self.assertEqual(output.status_code, 302)
- output = self.app.post(
- "/test/pull-request/close/1", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Overview - test - Pagure</title>", output_text
- )
- self.assertIn("Invalid input submitted", output_text)
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # Invalid project
- output = self.app.post(
- "/foo/pull-request/close/1", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # Invalid PR id
- output = self.app.post(
- "/test/pull-request/close/100",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Invalid user for this project
- output = self.app.post(
- "/test/pull-request/close/1", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/close/1", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # Project w/ pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/close/1", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Overview - test - Pagure</title>", output_text
- )
- self.assertIn("Pull request closed!", output_text)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email")
- def test_reopen_request_pull(self, send_email):
- """Test the reopen_request_pull endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=None,
- branch_from="feature",
- mtype="merge",
- )
- user = tests.FakeUser()
- with tests.user_set(self.app.application, user):
- output = self.app.post("/test/pull-request/1/reopen")
- self.assertEqual(output.status_code, 302)
- output = self.app.post(
- "/test/pull-request/1/reopen", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- #'Pull request reopened!',
- 'return window.confirm("Are you sure you want to reopen this requested pull?")',
- output_text,
- )
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # Invalid project
- output = self.app.post(
- "/foo/pull-request/1/reopen", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # Invalid PR id
- output = self.app.post(
- "/test/pull-request/100/reopen",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Invalid user for this project
- output = self.app.post(
- "/test/pull-request/1/reopen", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/reopen", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- # Project w/ pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = True
- settings["fedmsg_notifications"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- with testing.mock_sends(
- pagure_messages.PullRequestCommentAddedV1(
- topic="pagure.pull-request.comment.added",
- body={
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Closed",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been closed by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "agent": "pingou",
- },
- ),
- pagure_messages.PullRequestClosedV1(
- topic="pagure.pull-request.closed",
- body={
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "title": "PR from the feature branch",
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Closed",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been closed by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "merged": False,
- "agent": "pingou",
- },
- ),
- ):
- output = self.app.post(
- "/test/pull-request/close/1",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- with testing.mock_sends(
- pagure_messages.PullRequestCommentAddedV1(
- topic="pagure.pull-request.comment.added",
- body={
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "title": "PR from the feature branch",
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been closed by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- },
- {
- "id": 2,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been reopened by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- },
- ],
- },
- "agent": "pingou",
- },
- ),
- pagure_messages.PullRequestReopenedV1(
- topic="pagure.pull-request.reopened",
- body={
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "full_url": "http://localhost.localdomain/test",
- "fullname": "test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been closed by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- },
- {
- "id": 2,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "Pull-Request has been reopened by pingou",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- },
- ],
- },
- "agent": "pingou",
- },
- ),
- ):
- output = self.app.post(
- "/test/pull-request/1/reopen",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- 'return window.confirm("Are you sure you want to reopen this requested pull?")',
- output_text,
- )
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
- def test_update_pull_requests_assign(self):
- """Test the update_pull_requests endpoint when assigning a PR."""
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # No such project
- output = self.app.post("/foo/pull-request/1/update")
- self.assertEqual(output.status_code, 404)
- output = self.app.post("/test/pull-request/100/update")
- self.assertEqual(output.status_code, 404)
- # Invalid input
- output = self.app.post(
- "/test/pull-request/1/update", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertNotIn("Request assigned", output_text)
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"user": "pingou"}
- # No CSRF
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertNotIn("Request assigned", output_text)
- # Invalid assignee
- data = {"csrf_token": csrf_token, "user": "bar"}
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("No user "bar" found", output_text)
- # Assign the PR
- data = {"csrf_token": csrf_token, "user": "pingou"}
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- with testing.mock_sends(
- api.Message(
- topic="pagure.request.assigned.added",
- body={
- "request": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": {
- "name": "pingou",
- "fullname": "PY C",
- "full_url": "http://localhost.localdomain/user/pingou",
- "url_path": "user/pingou",
- },
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [],
- },
- "pullrequest": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "agent": "pingou",
- },
- ),
- pagure_messages.PullRequestAssignedAddedV1(
- topic="pagure.pull-request.assigned.added",
- body={
- "pullrequest": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "agent": "pingou",
- },
- ),
- ):
- output = self.app.post(
- "/test/pull-request/1/update",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("Request assigned", output_text)
- # Pull-Request closed - reset assignee
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- req = repo.requests[0]
- req.status = "Closed"
- req.closed_by_in = 1
- self.session.add(req)
- self.session.commit()
- data = {"csrf_token": csrf_token, "user": None}
- with testing.mock_sends(
- api.Message(
- topic="pagure.request.assigned.reset",
- body={
- "request": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Closed",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "**Metadata Update from @pingou**:\n- Request assigned",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Closed",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "**Metadata Update from @pingou**:\n- Request assigned",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "agent": "pingou",
- },
- ),
- pagure_messages.PullRequestAssignedResetV1(
- topic="pagure.pull-request.assigned.reset",
- body={
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Closed",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "**Metadata Update from @pingou**:\n- Request assigned",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "agent": "pingou",
- },
- ),
- ):
- output = self.app.post(
- "/test/pull-request/1/update",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
- def test_update_pull_requests_tag(self):
- """Test the update_pull_requests endpoint when tagging a PR."""
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- data = {"tag": "black"}
- # No CSRF
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertNotIn("Request assigned", output_text)
- # Tag the PR
- data = {"csrf_token": csrf_token, "tag": "black"}
- with testing.mock_sends(
- pagure_messages.PullRequestTagAddedV1(
- topic="pagure.pull-request.tag.added",
- body={
- # This is field is for backward compatibility but we
- # don't want to check it
- "pull_request": ANY,
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": ["black"],
- "comments": [],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "tags": ["black"],
- "agent": "pingou",
- },
- )
- ):
- output = self.app.post(
- "/test/pull-request/1/update",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("Pull-request tagged with: black", output_text)
- self.assertIn(
- 'title="comma separated list of tags"\n '
- 'value="black" />',
- output_text,
- )
- # Try as another user
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- # Tag the PR
- data = {"csrf_token": csrf_token, "tag": "blue, yellow"}
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- # Make the PR be from foo
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- req = repo.requests[0]
- req.user_id = 2
- self.session.add(req)
- self.session.commit()
- # Re-try to tag the PR
- data = {"csrf_token": csrf_token, "tag": "blue, yellow"}
- with testing.mock_sends(
- pagure_messages.PullRequestTagAddedV1(
- topic="pagure.pull-request.tag.added",
- body={
- "pull_request": ANY,
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "foo",
- "fullname": "foo bar",
- "url_path": "user/foo",
- "full_url": "http://localhost.localdomain/user/foo",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": ["black", "blue", "yellow"],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "**Metadata Update from "
- "@pingou**:\n- Pull-request tagged "
- "with: black",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "tags": ["blue", "yellow"],
- "agent": "foo",
- },
- ),
- pagure_messages.PullRequestTagRemovedV1(
- topic="pagure.pull-request.tag.removed",
- body={
- # This is field is for backward compatibility but we
- # don't want to check it
- "pull_request": ANY,
- "pullrequest": {
- "id": 1,
- "uid": ANY,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "foo",
- "fullname": "foo bar",
- "url_path": "user/foo",
- "full_url": "http://localhost.localdomain/user/foo",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": ["blue", "yellow"],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "**Metadata Update from "
- "@pingou**:\n- Pull-request tagged "
- "with: black",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": True,
- "reactions": {},
- }
- ],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "tags": ["black"],
- "agent": "foo",
- },
- ),
- ):
- output = self.app.post(
- "/test/pull-request/1/update",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- soup = BeautifulSoup(output.get_data(as_text=True), "html.parser")
- self.assertEqual(
- soup.find("title").string,
- "PR#1: PR from the feature branch - test\n - Pagure",
- )
- self.assertIn(
- "Pull-request **un**tagged with: black",
- output.get_data(as_text=True),
- )
- self.assertIn(
- "Pull-request tagged with: blue, yellow",
- output.get_data(as_text=True),
- )
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Pull-Request closed
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- req = repo.requests[0]
- req.status = "Closed"
- req.closed_by_in = 1
- self.session.add(req)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email")
- def test_fork_project(self, send_email):
- """Test the fork_project endpoint."""
- send_email.return_value = True
- tests.create_projects(self.session)
- for folder in ["docs", "tickets", "requests", "repos"]:
- tests.create_projects_git(
- os.path.join(self.path, folder), bare=True
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.post("/do_fork/test")
- self.assertEqual(output.status_code, 400)
- output = self.app.get("/new/")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "<strong>Create new Project</strong>",
- output.get_data(as_text=True),
- )
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/foo", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 404)
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.post("/do_fork/test")
- self.assertEqual(output.status_code, 400)
- data = {"csrf_token": csrf_token}
- with testing.mock_sends(
- pagure_messages.ProjectForkedV1(
- topic="pagure.project.forked",
- body={
- "project": {
- "id": 4,
- "name": "test",
- "fullname": "forks/foo/test",
- "url_path": "fork/foo/test",
- "full_url": "http://localhost.localdomain/fork/foo/test",
- "description": "test project #1",
- "namespace": None,
- "parent": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "foo",
- "fullname": "foo bar",
- "url_path": "user/foo",
- "full_url": "http://localhost.localdomain/user/foo",
- },
- "access_users": {
- "owner": ["foo"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [],
- "milestones": {},
- },
- "agent": "foo",
- },
- )
- ):
- output = self.app.post(
- "/do_fork/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
- def test_fork_project_non_master_default(self):
- """Test the fork_project endpoint with a project whose default branch
- is not master."""
- tests.create_projects(self.session)
- for folder in ["docs", "tickets", "requests", "repos"]:
- tests.create_projects_git(
- os.path.join(self.path, folder), bare=True
- )
- path = os.path.join(self.path, "repos", "test.git")
- tests.add_content_git_repo(path)
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- # Check before that the master branch is the default one - shown in the
- # default page
- output = self.app.get("/test")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<code class="py-1 px-2 font-weight-bold commit_branch">master</code><code',
- output_text,
- )
- # Create the main branch with some content and make it the default branch
- repo = pygit2.Repository(path)
- branchname = "main"
- repo.create_branch(branchname, repo.head.peel())
- pagure.lib.git.git_set_ref_head(project=project, branch=branchname)
- user = tests.FakeUser(username="foo")
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": self.get_csrf()}
- output = self.app.post(
- "/do_fork/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<code class="py-1 px-2 font-weight-bold commit_branch">main</code><code',
- output_text,
- )
- output = self.app.get("/fork/foo/test")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<code class="py-1 px-2 font-weight-bold commit_branch">main</code><code',
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_branch_space(self, send_email):
- """Test the new_request_pull endpoint."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/diff/master..foo bar")
- self.assertEqual(output.status_code, 400)
- output_text = output.get_data(as_text=True)
- self.assertIn("<p>Branch foo bar does not exist</p>", output_text)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull(self, send_email):
- """Test the new_request_pull endpoint."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/foo/diff/master..feature")
- self.assertEqual(output.status_code, 404)
- output = self.app.get("/test/diff/master..foo")
- self.assertEqual(output.status_code, 400)
- output = self.app.get("/test/diff/foo..master")
- self.assertEqual(output.status_code, 400)
- output = self.app.get("/test/diff/feature..master")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Diff from master to feature - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<p class="error"> No commits found </p>', output_text
- )
- output = self.app.get("/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Diff from feature to master - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertNotIn(
- '<input type="submit" class="submit positive button" '
- 'value="Create">',
- output_text,
- )
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- # Check that we prefilled the input fields as expected:
- self.assertIn(
- '<input class="form-control" id="title" name="title" '
- 'placeholder="Pull Request Title" required="required" '
- 'type="text" value="A commit on branch feature">',
- output_text,
- )
- self.assertIn(
- """<textarea class="form-control" rows=8 id="initial_comment" name="initial_comment"
- placeholder="Describe your changes" tabindex=1>
- More information</textarea>
- <div id="preview" class="p-1">""",
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="master"><span class="fa fa-random">'
- "</span> master</a>",
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Case 1 - Add an initial comment
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- with testing.mock_sends(
- pagure_messages.PullRequestNewV1(
- topic="pagure.pull-request.new",
- body={
- "pullrequest": {
- "id": 2,
- "full_url": "http://localhost.localdomain/test/pull-request/2",
- "uid": ANY,
- "title": "foo bar PR",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": "Test Initial Comment",
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [],
- },
- "agent": "pingou",
- },
- )
- ):
- output = self.app.post(
- "/test/diff/master..feature",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#2: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- self.assertEqual(output_text.count('title="PY C (pingou)"'), 2)
- # Test if the `open changed file icon` is displayed.
- self.assertIn(
- 'class="open_changed_file_icon_wrap"><span '
- 'class="fa fa-file-code-o fa-fw" '
- 'alt="Open changed file" title="Open changed file"></span>'
- "</a>",
- output_text,
- )
- # Case 2 - Add an empty initial comment
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "",
- }
- output = self.app.post(
- "/test/diff/master..feature", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#3: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertNotIn('<div id="comment-', output_text)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_filename_unicode(self, send_email):
- """Test the new_request_pull endpoint."""
- send_email.return_value = True
- # Create the main project in the DB
- item = pagure.lib.model.Project(
- user_id=1, # pingou
- name="test",
- description="test project #1",
- hook_token="aaabbbccc",
- )
- item.close_status = [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ]
- self.session.add(item)
- self.session.commit()
- # Create the fork
- item = pagure.lib.model.Project(
- user_id=1, # pingou
- name="test",
- description="test project #1",
- hook_token="aaabbbcccdd",
- parent_id=1,
- is_fork=True,
- )
- item.close_status = [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ]
- self.session.add(item)
- self.session.commit()
- # Create two git repos, one has 6 commits, the other 4 of which only
- # 1 isn't present in the first repo
- gitrepo = os.path.join(self.path, "repos", "test.git")
- pygit2.init_repository(gitrepo, bare=True)
- gitrepo2 = os.path.join(
- self.path, "repos", "forks", "pingou", "test.git"
- )
- pygit2.init_repository(gitrepo2, bare=True)
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- repopath = os.path.join(newpath, "test")
- clone_repo = pygit2.clone_repository(gitrepo, repopath)
- # Do 2 commits to the main repo
- for i in range(2):
- with open(os.path.join(repopath, "sources"), "w") as stream:
- stream.write("foo%s\n bar%s\n" % (i, i))
- clone_repo.index.add("sources")
- clone_repo.index.write()
- parents = []
- try:
- last_commit = clone_repo.revparse_single("HEAD")
- parents = [last_commit.oid.hex]
- except KeyError:
- pass
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature(
- "Cecil Committer", "cecil@committers.tld"
- )
- clone_repo.create_commit(
- "refs/heads/master", # the name of the reference to update
- author,
- committer,
- "Editing the file sources for testing #%s" % i,
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- parents,
- )
- # Push to the main repo
- refname = "refs/heads/master:refs/heads/master"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- # Push to the fork repo
- remote = clone_repo.remotes.create("pingou_fork", gitrepo2)
- PagureRepo.push(remote, refname)
- # Add 1 commits to the fork repo
- repopath = os.path.join(newpath, "pingou_test")
- clone_repo = pygit2.clone_repository(gitrepo2, repopath)
- with open(os.path.join(repopath, "soürces"), "w") as stream:
- stream.write("foo\n bar\n")
- clone_repo.index.add("soürces")
- clone_repo.index.write()
- with open(os.path.join(repopath, "fóß"), "w") as stream:
- stream.write("foo\n bar\n")
- clone_repo.index.add("fóß")
- clone_repo.index.write()
- last_commit = clone_repo.revparse_single("HEAD")
- # Commits the files added
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- last_commit = clone_repo.create_commit(
- "refs/heads/feature_foo", # the name of the reference to update
- author,
- committer,
- "New edition on side branch of the file sources for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [last_commit.oid.hex],
- )
- # Push to the fork repo
- ori_remote = clone_repo.remotes[0]
- refname = "refs/heads/feature_foo:refs/heads/feature_foo"
- PagureRepo.push(ori_remote, refname)
- shutil.rmtree(newpath)
- # Create the PR between the two repos
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- forked_repo = pagure.lib.query.get_authorized_project(
- self.session, "test", user="pingou"
- )
- req = pagure.lib.query.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from="feature_foo",
- repo_to=repo,
- branch_to="master",
- title="test pull-request",
- user="pingou",
- )
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, "test pull-request")
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- output = self.app.get("/fork/pingou/test/diff/master..feature_foo")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - fork/pingou/test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- # Check that we prefilled the input fields as expected:
- self.assertIn(
- '<input class="form-control" id="title" name="title" '
- 'placeholder="Pull Request Title" required="required" '
- 'type="text" value="New edition on side branch of the file '
- 'sources for testing">',
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="master"><span class="fa fa-random">'
- "</span> master</a>",
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Case 1 - Add an initial comment
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/fork/pingou/test/diff/master..feature_foo",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#2: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- self.assertEqual(output_text.count('title="PY C (pingou)"'), 2)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_req_sign_off_view(self, send_email):
- """Test the new_request_pull endpoint."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- # Enforce Signed-of-by in the repo
- settings = repo.settings
- settings["Enforce_signed-off_commits_in_pull-request"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Diff from feature to master - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- "This project enforces the "
- "Signed-off-by statement on all commits",
- output_text,
- )
- self.assertNotIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- self.assertNotIn(
- "This repo enforces that "
- "all commits are signed off by their author.",
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_req_sign_off_submit(self, send_email):
- """Test the new_request_pull endpoint."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- # Enforce Signed-of-by in the repo
- settings = repo.settings
- settings["Enforce_signed-off_commits_in_pull-request"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- "This project enforces the "
- "Signed-off-by statement on all commits",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Try to create the PR
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/test/diff/master..feature", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - test\n - "
- "Pagure</title>",
- output_text,
- )
- # Flashed information message
- self.assertIn(
- "This project enforces the "
- "Signed-off-by statement on all commits",
- output_text,
- )
- # Flashed error message
- self.assertIn(
- "This repo enforces that "
- "all commits are signed off by their author.",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_request_pull_commit_start_stop(self, send_email):
- """Test the the commit start and stop of brand new PR."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Case 1 - Add an initial comment
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/test/diff/master..feature", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#2: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- # Check if commit start and stop have been set for PR#2
- request = pagure.lib.query.search_pull_requests(
- self.session, project_id=1, requestid=2
- )
- self.assertIsNotNone(request.commit_start)
- self.assertIsNotNone(request.commit_stop)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_from_fork_branch(self, send_email):
- """Test creating a fork to fork PR."""
- send_email.return_value = True
- # Create main repo with some content
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- tests.add_content_git_repo(
- os.path.join(self.path, "repos", "test.git")
- )
- # Create fork repo with more content
- tests.create_projects(
- self.session, is_fork=True, hook_token_suffix="fork"
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "pingou"), bare=True
- )
- tests.add_content_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git")
- )
- tests.add_readme_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git"),
- branch="feature",
- )
- tests.add_readme_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git"),
- branch="random_branch",
- )
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": self.get_csrf()}
- output = self.app.post(
- "/do_fork/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that Ralph's fork do exist
- output = self.app.get("/fork/pingou/test")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- # Try opening a pull-request
- output = self.app.get("/fork/pingou/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - "
- "fork/pingou/test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="master"><span class="fa fa-random">'
- "</span> master</a>",
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="random_branch"><span class="fa fa-random">'
- "</span> random_branch</a>",
- output_text,
- )
- @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
- def test_new_request_pull_from_fork_fixing_ticket(self):
- """Test creating a fork to fork PR fixing a ticket."""
- # Create main repo with some content
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- tests.add_content_git_repo(
- os.path.join(self.path, "repos", "test.git")
- )
- # Create fork repo with more content
- tests.create_projects(
- self.session, is_fork=True, hook_token_suffix="fork"
- )
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "pingou"), bare=True
- )
- tests.add_content_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git")
- )
- tests.add_readme_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git"),
- branch="feature",
- )
- tests.add_readme_git_repo(
- os.path.join(self.path, "repos", "forks", "pingou", "test.git"),
- branch="random_branch",
- )
- # Check relations before we create the PR
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 0)
- self.assertEqual(len(project.issues), 0)
- # Create issues to link to
- msg = pagure.lib.query.new_issue(
- session=self.session,
- repo=project,
- title="tést íssüé",
- content="We should work on this",
- user="pingou",
- )
- self.session.commit()
- self.assertEqual(msg.title, "tést íssüé")
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- csrf_token = self.get_csrf()
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that pingou's fork do exist
- output = self.app.get("/fork/pingou/test")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- prid=2,
- )
- # Try opening a pull-request
- output = self.app.get("/fork/pingou/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - "
- "fork/pingou/test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="master"><span class="fa fa-random">'
- "</span> master</a>",
- output_text,
- )
- self.assertIn(
- '<a class="dropdown-item branch_from_item pointer" '
- 'data-value="random_branch"><span class="fa fa-random">'
- "</span> random_branch</a>",
- output_text,
- )
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment\n\nFixes #1",
- }
- output = self.app.post(
- "/fork/pingou/test/diff/master..feature",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#3: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn(
- "<p>Test Initial Comment</p>\n<p>Fixes <a href", output_text
- )
- project = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(len(project.requests), 2)
- self.assertEqual(len(project.requests[0].related_issues), 0)
- self.assertEqual(len(project.requests[1].related_issues), 1)
- self.assertEqual(len(project.issues), 1)
- self.assertEqual(len(project.issues[0].related_prs), 1)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_fork_to_fork_pr_disabled(self, send_email):
- """Test creating a fork to fork PR."""
- send_email.return_value = True
- self.test_fork_project()
- # Create a 3rd user
- item = pagure.lib.model.User(
- user="ralph",
- fullname="Ralph bar",
- password="ralph_foo",
- default_email="ralph@bar.com",
- )
- self.session.add(item)
- item = pagure.lib.model.UserEmail(user_id=3, email="ralph@bar.com")
- self.session.add(item)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "ralph"
- with tests.user_set(self.app.application, user):
- # Have Ralph fork, foo's fork of test
- output = self.app.get("/fork/foo/test")
- self.assertEqual(output.status_code, 200)
- output = self.app.post("/do_fork/fork/foo/test")
- self.assertEqual(output.status_code, 400)
- csrf_token = self.get_csrf()
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/fork/foo/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that Ralph's fork do exist
- output = self.app.get("/fork/ralph/test")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- )
- # Try opening a pull-request
- output = self.app.get("/fork/ralph/test/diff/master..feature")
- self.assertEqual(output.status_code, 404)
- self.assertIn(
- "<p>No pull-request allowed on this project</p>",
- output.get_data(as_text=True),
- )
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_fork_to_fork(self, send_email):
- """Test creating a fork to fork PR."""
- send_email.return_value = True
- self.test_fork_project()
- # Create a 3rd user
- item = pagure.lib.model.User(
- user="ralph",
- fullname="Ralph bar",
- password="ralph_foo",
- default_email="ralph@bar.com",
- )
- self.session.add(item)
- item = pagure.lib.model.UserEmail(user_id=3, email="ralph@bar.com")
- self.session.add(item)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "ralph"
- with tests.user_set(self.app.application, user):
- # Have Ralph fork, foo's fork of test
- output = self.app.get("/fork/foo/test")
- self.assertEqual(output.status_code, 200)
- output = self.app.post("/do_fork/fork/foo/test")
- self.assertEqual(output.status_code, 400)
- csrf_token = self.get_csrf()
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/fork/foo/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that Ralph's fork do exist
- output = self.app.get("/fork/ralph/test")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- # Turn on pull-request on the fork
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- settings = repo.settings
- settings["pull_requests"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- # Add some content to the parent
- set_up_git_repo(
- self.session,
- self.path,
- new_project=repo,
- branch_from="master",
- mtype="FF",
- name_from=repo.fullname,
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- prid=2,
- name_from=fork.fullname,
- )
- # Try opening a pull-request
- output = self.app.get("/fork/ralph/test/diff/master..feature")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - fork/ralph/test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Case 1 - Add an initial comment
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/fork/ralph/test/diff/master..feature",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: foo bar PR - fork/foo/test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_fork_to_other_fork(self, send_email):
- """Test creating a PR from fork to a fork of the same family."""
- send_email.return_value = True
- self.test_fork_project()
- # Create a 3rd user
- item = pagure.lib.model.User(
- user="ralph",
- fullname="Ralph bar",
- password="ralph_foo",
- default_email="ralph@bar.com",
- )
- self.session.add(item)
- item = pagure.lib.model.UserEmail(user_id=3, email="ralph@bar.com")
- self.session.add(item)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "ralph"
- with tests.user_set(self.app.application, user):
- csrf_token = self.get_csrf()
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/test", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that Ralph's fork do exist
- output = self.app.get("/fork/ralph/test")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- # Turn on pull-request on the fork
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- settings = repo.settings
- settings["pull_requests"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- # Add some content to the parents
- set_up_git_repo(
- self.session,
- self.path,
- new_project=repo,
- branch_from="master",
- mtype="FF",
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=repo,
- branch_from="master",
- mtype="FF",
- name_from=repo.fullname,
- prid=2,
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- prid=3,
- name_from=fork.fullname,
- )
- # Try opening a pull-request
- output = self.app.get(
- "/fork/ralph/test/diff/master..feature?project_to=fork/foo/test"
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Create new Pull Request for master - fork/ralph/test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<input type="submit" class="btn btn-primary" value="Create Pull Request">\n',
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- # Case 1 - Opening PR to fork/foo/test
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/fork/ralph/test/diff/master..feature?project_to=fork/foo/test",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: foo bar PR - fork/foo/test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- # Case 1 - Opening PR to parent repo, shows project_to works
- output = self.app.post(
- "/fork/ralph/test/diff/master..feature",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#4: foo bar PR - test\n - Pagure</title>",
- output_text,
- )
- self.assertIn("<p>Test Initial Comment</p>", output_text)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_fork_to_other_unrelated_fork(self, send_email):
- """Test creating a PR from fork to fork that isn't from the same
- family.
- """
- send_email.return_value = True
- self.test_fork_project()
- # Create a 3rd user
- item = pagure.lib.model.User(
- user="ralph",
- fullname="Ralph bar",
- password="ralph_foo",
- default_email="ralph@bar.com",
- )
- self.session.add(item)
- item = pagure.lib.model.UserEmail(user_id=3, email="ralph@bar.com")
- self.session.add(item)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "ralph"
- with tests.user_set(self.app.application, user):
- csrf_token = self.get_csrf()
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/do_fork/test2", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- # Check that Ralph's fork do exist
- output = self.app.get("/fork/ralph/test2")
- self.assertEqual(output.status_code, 200)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- # Turn on pull-request on the fork
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- settings = repo.settings
- settings["pull_requests"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- # Add some content to the parent
- set_up_git_repo(
- self.session,
- self.path,
- new_project=repo,
- branch_from="master",
- mtype="FF",
- name_from=repo.fullname,
- )
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test2", user="ralph"
- )
- set_up_git_repo(
- self.session,
- self.path,
- new_project=fork,
- branch_from="feature",
- mtype="FF",
- prid=2,
- name_from=fork.fullname,
- )
- # Case 1 - Opening PR to fork/foo/test
- data = {
- "csrf_token": csrf_token,
- "title": "foo bar PR",
- "initial_comment": "Test Initial Comment",
- }
- output = self.app.post(
- "/fork/ralph/test2/diff/master..feature?project_to=fork/foo/test",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 400)
- self.assertIn(
- "<p>fork/foo/test is not part of fork/ralph/test2's "
- "family</p>",
- output.get_data(as_text=True)
- .replace("'", "'")
- .replace("'", "'"),
- )
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_empty_repo(self, send_email):
- """Test the new_request_pull endpoint against an empty repo."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.get(
- "/fork/foo/test/diff/master..feature", follow_redirects=True
- )
- self.assertEqual(output.status_code, 400)
- self.assertIn(
- "<p>Fork is empty, there are no commits to create a pull "
- "request with</p>",
- output.get_data(as_text=True),
- )
- output = self.app.get("/test/new_issue")
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token, "title": "foo bar PR"}
- output = self.app.post(
- "/test/diff/master..feature", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 400)
- self.assertIn(
- "<p>Fork is empty, there are no commits to create a pull "
- "request with</p>",
- output.get_data(as_text=True),
- )
- shutil.rmtree(newpath)
- @patch("pagure.lib.notify.send_email")
- def test_new_request_pull_empty_fork(self, send_email):
- """Test the new_request_pull endpoint against an empty repo."""
- send_email.return_value = True
- self.test_fork_project()
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- fork = pagure.lib.query.get_authorized_project(
- self.session, "test", user="foo"
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.get(
- "/fork/foo/test/diff/master..master", follow_redirects=True
- )
- self.assertEqual(output.status_code, 400)
- self.assertIn(
- "<p>Fork is empty, there are no commits to create a pull "
- "request with</p>",
- output.get_data(as_text=True),
- )
- shutil.rmtree(newpath)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email")
- def test_pull_request_add_comment(self, send_email):
- """Test the pull_request_add_comment endpoint."""
- send_email.return_value = True
- self.test_request_pull()
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.post("/foo/pull-request/1/comment")
- self.assertEqual(output.status_code, 404)
- output = self.app.post("/test/pull-request/100/comment")
- self.assertEqual(output.status_code, 404)
- output = self.app.post("/test/pull-request/1/comment")
- self.assertEqual(output.status_code, 200)
- self.assertTrue(
- output.get_data(as_text=True).startswith(
- '\n<section class="add_comment">'
- )
- )
- csrf_token = self.get_csrf(output=output)
- data = {
- "csrf_token": csrf_token,
- "comment": "This look alright but we can do better",
- }
- with testing.mock_sends(
- pagure_messages.PullRequestCommentAddedV1(
- topic="pagure.pull-request.comment.added",
- body={
- "pullrequest": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [
- {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "This look alright but we can do better",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": None,
- "editor": None,
- "notification": False,
- "reactions": {},
- }
- ],
- },
- "agent": "pingou",
- },
- )
- ):
- output = self.app.post(
- "/test/pull-request/1/comment",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn("Comment added", output_text)
- self.assertEqual(output_text.count('title="PY C (pingou)"'), 2)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/comment",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- def test_pull_request_drop_comment(self, send_email):
- """Test the pull_request_drop_comment endpoint."""
- send_email.return_value = True
- self.test_pull_request_add_comment()
- # Project w/ pull-request
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["pull_requests"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- output = self.app.post("/foo/pull-request/1/comment/drop")
- self.assertEqual(output.status_code, 404)
- output = self.app.post("/test/pull-request/100/comment/drop")
- self.assertEqual(output.status_code, 404)
- output = self.app.post(
- "/test/pull-request/1/comment/drop", follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- # self.assertIn('href="#comment-1">¶</a>', output_text)
- self.assertIn(
- "<p>This look alright but we can do better</p>", output_text
- )
- csrf_token = self.get_csrf(output=output)
- # Invalid comment id
- data = {"csrf_token": csrf_token, "drop_comment": "10"}
- output = self.app.post(
- "/test/pull-request/1/comment/drop",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- data["drop_comment"] = "1"
- output = self.app.post(
- "/test/pull-request/1/comment/drop",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Drop comment
- output = self.app.post(
- "/test/pull-request/1/comment/drop",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("Comment removed", output_text)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/comment/drop",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- @patch.dict(
- "pagure.config.config", {"FEDORA_MESSAGING_NOTIFICATIONS": True}
- )
- @patch("pagure.lib.notify.send_email")
- def test_pull_request_edit_comment(self, send_email):
- """Test the pull request edit comment endpoint"""
- send_email.return_value = True
- self.test_request_pull()
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Repo 'foo' does not exist so it is verifying that condition
- output = self.app.post("/foo/pull-request/1/comment/1/edit")
- self.assertEqual(output.status_code, 404)
- # Here no comment is present in the PR so its verifying that condition
- output = self.app.post("/test/pull-request/100/comment/100/edit")
- self.assertEqual(output.status_code, 404)
- output = self.app.post("/test/pull-request/1/comment")
- self.assertEqual(output.status_code, 200)
- # Creating comment to play with
- self.assertTrue(
- output.get_data(as_text=True).startswith(
- '\n<section class="add_comment">'
- )
- )
- csrf_token = self.get_csrf(output=output)
- data = {
- "csrf_token": csrf_token,
- "comment": "This look alright but we can do better",
- }
- output = self.app.post(
- "/test/pull-request/1/comment",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn("Comment added", output_text)
- # Check if the comment is there
- self.assertIn(
- "<p>This look alright but we can do better</p>", output_text
- )
- output = self.app.get("/test/pull-request/1/comment/1/edit")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn('<section class="edit_comment">', output_text)
- # Checking if the comment is there in the update page
- self.assertIn(
- "This look alright but we can do better</textarea>",
- output_text,
- )
- csrf_token = self.get_csrf(output=output)
- data = {
- "csrf_token": csrf_token,
- "update_comment": "This look alright but we can do better than this.",
- }
- with testing.mock_sends(
- pagure_messages.PullRequestCommentEditedV1(
- topic="pagure.pull-request.comment.edited",
- body={
- "pullrequest": {
- "id": 1,
- "full_url": "http://localhost.localdomain/test/pull-request/1",
- "uid": ANY,
- "title": "PR from the feature branch",
- "branch": "master",
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "branch_from": "feature",
- "repo_from": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "full_url": "http://localhost.localdomain/test",
- "url_path": "test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "remote_git": None,
- "date_created": ANY,
- "updated_on": ANY,
- "last_updated": ANY,
- "closed_at": None,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "assignee": None,
- "status": "Open",
- "commit_start": ANY,
- "commit_stop": ANY,
- "closed_by": None,
- "initial_comment": None,
- "cached_merge_status": "unknown",
- "threshold_reached": None,
- "tags": [],
- "comments": [],
- },
- "project": {
- "id": 1,
- "name": "test",
- "fullname": "test",
- "url_path": "test",
- "full_url": "http://localhost.localdomain/test",
- "description": "test project #1",
- "namespace": None,
- "parent": None,
- "date_created": ANY,
- "date_modified": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "access_users": {
- "owner": ["pingou"],
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "access_groups": {
- "admin": [],
- "commit": [],
- "collaborator": [],
- "ticket": [],
- },
- "tags": [],
- "priorities": {},
- "custom_keys": [],
- "close_status": [
- "Invalid",
- "Insufficient data",
- "Fixed",
- "Duplicate",
- ],
- "milestones": {},
- },
- "comment": {
- "id": 1,
- "commit": None,
- "tree": None,
- "filename": None,
- "line": None,
- "comment": "This look alright but we can do better than this.",
- "parent": None,
- "date_created": ANY,
- "user": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "edited_on": ANY,
- "editor": {
- "name": "pingou",
- "fullname": "PY C",
- "url_path": "user/pingou",
- "full_url": "http://localhost.localdomain/user/pingou",
- },
- "notification": False,
- "reactions": {},
- },
- "agent": "pingou",
- },
- )
- ):
- output = self.app.post(
- "/test/pull-request/1/comment/1/edit",
- data=data,
- follow_redirects=True,
- )
- output_text = output.get_data(as_text=True)
- # Checking if the comment is updated in the main page
- self.assertIn(
- "<p>This look alright but we can do better than this.</p>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- # Checking if Edited by User is there or not
- pattern = (
- re.escape("<small>Edited ")
- + '<span title="[0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:'
- + '[0-9]{2} UTC" data-toggle="tooltip">(just now|seconds ago)</span>'
- + re.escape(" by pingou </small>")
- )
- self.assertIsNotNone(re.search(pattern, output_text))
- self.assertIn("Comment updated", output_text)
- # Project w/o pull-request
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["pull_requests"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post(
- "/test/pull-request/1/comment/edit/1",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- @patch("pagure.lib.notify.send_email")
- def test_merge_request_pull_FF_w_merge_commit(self, send_email):
- """Test the merge_request_pull endpoint with a FF PR but with a
- merge commit.
- """
- send_email.return_value = True
- self.test_request_pull()
- user = tests.FakeUser()
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- csrf_token = self.get_csrf(output=output)
- # No CSRF
- output = self.app.post(
- "/test/pull-request/1/merge", data={}, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n - "
- "Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<h4 class="ml-1">\n <div>\n '
- '<span class="fa fa-fw text-success fa-arrow-circle-down pt-1"></span>\n '
- '<span class="text-success '
- 'font-weight-bold">#1</span>\n '
- '<span class="font-weight-bold">\n '
- "PR from the feature branch\n",
- output_text,
- )
- self.assertIn(
- 'title="View file as of 2a552bb">sources</a>', output_text
- )
- # Wrong project
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/foobar/pull-request/100/merge",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Wrong project
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 403)
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Wrong request id
- data = {"csrf_token": csrf_token}
- output = self.app.post(
- "/test/pull-request/100/merge",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- # Project requiring a merge commit
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- settings = repo.settings
- settings["always_merge"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- # Merge
- output = self.app.post(
- "/test/pull-request/1/merge", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output = self.app.get("/test/commits")
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Commits - test - Pagure</title>", output_text
- )
- self.assertIn("Merge #1 `PR from the feature branch`", output_text)
- self.assertIn("A commit on branch feature", output_text)
- # Check if the closing notification was added
- output = self.app.get("/test/pull-request/1")
- output_text = output.get_data(as_text=True)
- self.assertIsNotNone(re.search(MERGED_PATTERN, output_text))
- @patch("pagure.lib.notify.send_email")
- def test_internal_endpoint_main_ahead(self, send_email):
- """Test the new_request_pull endpoint when the main repo is ahead
- of the fork.
- """
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- gitrepo = os.path.join(self.path, "repos", "test.git")
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Make the main repo be ahead of the fork
- # First commit
- newpath = tempfile.mkdtemp(prefix="pagure-test")
- repopath = os.path.join(newpath, "test")
- clone_repo = pygit2.clone_repository(gitrepo, repopath)
- # Create a file in that git repo
- with open(os.path.join(repopath, "testfile"), "w") as stream:
- stream.write("foo\n bar")
- clone_repo.index.add("testfile")
- clone_repo.index.write()
- # Commits the files added
- last_commit = clone_repo.revparse_single("HEAD")
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- clone_repo.create_commit(
- "refs/heads/master", # the name of the reference to update
- author,
- committer,
- "Add testfile file for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [last_commit.oid.hex],
- )
- # Second commit
- with open(os.path.join(repopath, "testfile"), "a") as stream:
- stream.write("\nfoo2\n bar2")
- clone_repo.index.add("testfile")
- clone_repo.index.write()
- # Commits the files added
- last_commit = clone_repo.revparse_single("HEAD")
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- clone_repo.create_commit(
- "refs/heads/master", # the name of the reference to update
- author,
- committer,
- "Add a second commit to testfile for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [last_commit.oid.hex],
- )
- # Third commit
- with open(os.path.join(repopath, "testfile"), "a") as stream:
- stream.write("\nfoo3\n bar3")
- clone_repo.index.add("testfile")
- clone_repo.index.write()
- # Commits the files added
- last_commit = clone_repo.revparse_single("HEAD")
- tree = clone_repo.index.write_tree()
- author = pygit2.Signature("Alice Author", "alice@authors.tld")
- committer = pygit2.Signature("Cecil Committer", "cecil@committers.tld")
- clone_repo.create_commit(
- "refs/heads/master", # the name of the reference to update
- author,
- committer,
- "Add a third commit to testfile for testing",
- # binary string representing the tree object ID
- tree,
- # list of binary strings representing parents of the new commit
- [last_commit.oid.hex],
- )
- refname = "refs/heads/master:refs/heads/master"
- ori_remote = clone_repo.remotes[0]
- PagureRepo.push(ori_remote, refname)
- shutil.rmtree(newpath)
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- csrf_token = self.get_csrf()
- output = self.app.post(
- "/pv/pull-request/ready",
- data={"repo": "test", "csrf_token": csrf_token},
- )
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.get_data(as_text=True))
- self.assertEqual(sorted(data.keys()), ["code", "task"])
- self.assertEqual(data["code"], "OK")
- @patch("pagure.lib.notify.send_email")
- def test_fork_edit_file(self, send_email):
- """Test the fork_edit file endpoint."""
- send_email.return_value = True
- # Git repo not found
- output = self.app.post("fork_edit/test/edit/master/f/sources")
- self.assertEqual(output.status_code, 404)
- tests.create_projects(self.session)
- for folder in ["docs", "tickets", "requests", "repos"]:
- tests.create_projects_git(
- os.path.join(self.path, folder), bare=True
- )
- # User not logged in
- output = self.app.post("fork_edit/test/edit/master/f/sources")
- self.assertEqual(output.status_code, 302)
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Invalid request
- output = self.app.post("fork_edit/test/edit/master/f/source")
- self.assertEqual(output.status_code, 400)
- output = self.app.get("/new/")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "<strong>Create new Project</strong>",
- output.get_data(as_text=True),
- )
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # No files can be found since they are not added
- output = self.app.post(
- "fork_edit/test/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": csrf_token}
- # Invalid request
- output = self.app.post(
- "fork_edit/test/edit/master/f/sources", follow_redirects=True
- )
- self.assertEqual(output.status_code, 400)
- # Add content to the repo
- tests.add_content_git_repo(
- os.path.join(pagure.config.config["GIT_FOLDER"], "test.git")
- )
- tests.add_readme_git_repo(
- os.path.join(pagure.config.config["GIT_FOLDER"], "test.git")
- )
- tests.add_binary_git_repo(
- os.path.join(pagure.config.config["GIT_FOLDER"], "test.git"),
- "test.jpg",
- )
- # Check if button exists
- output = self.app.get("/test/blob/master/f/sources")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Fork and Edit\n </button>\n",
- output.get_data(as_text=True),
- )
- # Check fork-edit doesn't show for binary files
- output = self.app.get("/test/blob/master/f/test.jpg")
- self.assertEqual(output.status_code, 200)
- self.assertNotIn(b"<html", output.data)
- # Check for edit panel
- output = self.app.post(
- "fork_edit/test/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<li><a href="/fork/foo/test/tree/master">'
- '<span class="fa fa-random"></span> master</a>'
- '</li><li class="active"><span class="fa fa-file">'
- "</span> sources</li>",
- output_text,
- )
- self.assertIn(
- '<textarea id="textareaCode" name="content">foo\n bar</textarea>',
- output_text,
- )
- # Check for edit panel- Fork already done
- output = self.app.post(
- "fork_edit/test/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn("<title>Edit - test - Pagure</title>", output_text)
- self.assertIn(
- "You had already forked " "this project", output_text
- )
- self.assertIn(
- '<i class="fa fa-code-fork fa-fw"></i> View Upstream',
- output_text,
- )
- self.assertIn(
- '<li><a href="/fork/foo/test/tree/master">'
- '<span class="fa fa-random"></span> master</a>'
- '</li><li class="active"><span class="fa fa-file">'
- "</span> sources</li>",
- output_text,
- )
- self.assertIn(
- '<textarea id="textareaCode" name="content">foo\n bar</textarea>',
- output_text,
- )
- # View what's supposed to be an image
- output = self.app.post(
- "fork_edit/test/edit/master/f/test.jpg",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 400)
- self.assertIn(
- b"<p>Cannot edit binary files</p>",
- output.data,
- )
- # Check fork-edit shows when user is not logged in
- output = self.app.get("/test/blob/master/f/sources")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Fork and Edit\n </button>\n",
- output.get_data(as_text=True),
- )
- # Check if fork-edit shows for different user
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Check if button exists
- output = self.app.get("/test/blob/master/f/sources")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Edit in your fork\n </button>\n",
- output.get_data(as_text=True),
- )
- # Check fork-edit doesn't show for binary
- output = self.app.get("/test/blob/master/f/test.jpg")
- self.assertEqual(output.status_code, 200)
- self.assertNotIn(b"<html", output.data)
- @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
- def test_fork_edit_file_namespace(self):
- """Test the fork_edit file endpoint on a namespaced project."""
- tests.create_projects(self.session)
- for folder in ["docs", "tickets", "requests", "repos"]:
- tests.create_projects_git(
- os.path.join(self.path, folder), bare=True
- )
- # User not logged in
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources"
- )
- self.assertEqual(output.status_code, 302)
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Invalid request
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources"
- )
- self.assertEqual(output.status_code, 400)
- output = self.app.get("/new/")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "<strong>Create new Project</strong>",
- output.get_data(as_text=True),
- )
- csrf_token = self.get_csrf(output=output)
- data = {"csrf_token": csrf_token}
- # No files can be found since they are not added
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 404)
- user = tests.FakeUser()
- user.username = "foo"
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": csrf_token}
- # Invalid request
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources",
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 400)
- # Add content to the repo
- tests.add_content_git_repo(
- os.path.join(
- pagure.config.config["GIT_FOLDER"],
- "somenamespace",
- "test3.git",
- )
- )
- tests.add_readme_git_repo(
- os.path.join(
- pagure.config.config["GIT_FOLDER"],
- "somenamespace",
- "test3.git",
- )
- )
- tests.add_binary_git_repo(
- os.path.join(
- pagure.config.config["GIT_FOLDER"],
- "somenamespace",
- "test3.git",
- ),
- "test.jpg",
- )
- # Check if button exists
- output = self.app.get("/somenamespace/test3/blob/master/f/sources")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Fork and Edit\n </button>\n",
- output.get_data(as_text=True),
- )
- # Check fork-edit doesn't show for binary files
- output = self.app.get(
- "/somenamespace/test3/blob/master/f/test.jpg"
- )
- self.assertEqual(output.status_code, 200)
- self.assertNotIn(b"<html", output.data)
- # Check for edit panel
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Edit - somenamespace/test3 - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-code-fork fa-fw"></i> View Upstream',
- output_text,
- )
- self.assertIn(
- '<li><a href="/fork/foo/somenamespace/test3/tree/master">'
- '<span class="fa fa-random"></span> master</a>'
- '</li><li class="active"><span class="fa fa-file">'
- "</span> sources</li>",
- output_text,
- )
- self.assertIn(
- '<textarea id="textareaCode" name="content">foo\n bar</textarea>',
- output_text,
- )
- # Check for edit panel - while the project was already forked
- output = self.app.post(
- "fork_edit/somenamespace/test3/edit/master/f/sources",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Edit - somenamespace/test3 - Pagure</title>",
- output_text,
- )
- self.assertIn(
- "You had already forked " "this project", output_text
- )
- self.assertIn(
- '<i class="fa fa-code-fork fa-fw"></i> View Upstream',
- output_text,
- )
- self.assertIn(
- '<li><a href="/fork/foo/somenamespace/test3/tree/master">'
- '<span class="fa fa-random"></span> master</a>'
- '</li><li class="active"><span class="fa fa-file">'
- "</span> sources</li>",
- output_text,
- )
- self.assertIn(
- '<textarea id="textareaCode" name="content">foo\n bar</textarea>',
- output_text,
- )
- @patch("pagure.lib.notify.send_email")
- def test_fork_without_main_repo(self, send_email):
- """Test the fork without the main repo."""
- send_email.return_value = True
- tests.create_projects(self.session)
- # Create a fork with no parent i.e parent_id = None
- item = pagure.lib.model.Project(
- user_id=2, # foo
- name="test",
- description="test project #1",
- hook_token="aaabbb",
- is_fork=True,
- parent_id=None,
- )
- self.session.add(item)
- self.session.commit()
- # Get fork project
- project = pagure.lib.query._get_project(self.session, "test", "foo")
- # Pull-requests and issue-trackers are off for forks
- # lib function is not used here so mannually turning them off
- project_settings = project.settings
- project_settings["pull_requests"] = False
- project_settings["issue_tracker"] = False
- project.settings = project_settings
- self.session.add(project)
- self.session.commit()
- tests.create_projects_git(
- os.path.join(self.path, "repos", "forks", "foo"), bare=True
- )
- # Create a git repo to play with
- gitrepo = os.path.join(self.path, "repos", "test.git")
- self.assertFalse(os.path.exists(gitrepo))
- os.makedirs(gitrepo)
- repo = pygit2.init_repository(gitrepo, bare=True)
- # Create a fork of this repo
- newpath = tempfile.mkdtemp(prefix="pagure-fork-test")
- gitrepo = os.path.join(self.path, "repos", "forks", "foo", "test.git")
- new_repo = pygit2.clone_repository(gitrepo, newpath)
- tests.add_content_git_repo(gitrepo)
- # UI test for deleted main
- output = self.app.get("/fork/foo/test")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Forked from a deleted repository", output.get_data(as_text=True)
- )
- # Testing commit endpoint
- output = self.app.get("/fork/foo/test/commits/master")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- 'Commits <span class="badge badge-secondary"> 2</span>\n',
- output.get_data(as_text=True),
- )
- # Test pull-request endpoint
- output = self.app.get("/fork/foo/test/pull-requests")
- self.assertEqual(output.status_code, 404)
- # Test issue-tracker endpoint
- output = self.app.get("/fork/foo/test/issues")
- self.assertEqual(output.status_code, 404)
- shutil.rmtree(newpath)
- def _set_up_for_reaction_test(self):
- self.session.add(
- pagure.lib.model.User(
- user="jdoe",
- fullname="John Doe",
- password=b"password",
- default_email="jdoe@example.com",
- )
- )
- self.session.commit()
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- pagure.lib.query.get_authorized_project(self.session, "test")
- request = pagure.lib.query.search_pull_requests(
- self.session, requestid=1, project_id=1
- )
- pagure.lib.query.add_pull_request_comment(
- self.session,
- request=request,
- commit=None,
- tree_id=None,
- filename=None,
- row=None,
- comment="Hello",
- user="jdoe",
- )
- self.session.commit()
- @patch("pagure.lib.notify.send_email")
- def test_add_reaction(self, send_email):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- self._set_up_for_reaction_test()
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- data = {
- "csrf_token": self.get_csrf(output=output),
- "reaction": "Thumbs up",
- }
- output = self.app.post(
- "/test/pull-request/1/comment/1/react",
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- # Load the page and check reaction is added.
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- self.assertIn(
- "Thumbs up sent by pingou", output.get_data(as_text=True)
- )
- @patch("pagure.lib.notify.send_email")
- def test_add_reaction_unauthenticated(self, send_email):
- """Test the request_pull endpoint."""
- send_email.return_value = True
- self._set_up_for_reaction_test()
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- data = {
- "csrf_token": self.get_csrf(output=output),
- "reaction": "Thumbs down",
- }
- output = self.app.post(
- "/test/pull-request/1/comment/1/react",
- data=data,
- follow_redirects=False,
- )
- # Redirect to login page
- self.assertEqual(output.status_code, 302)
- self.assertIn("/login/", output.headers["Location"])
- class TestTicketAccessEditPRMetadata(tests.Modeltests):
- """Tests that people with ticket access on a project can edit the
- meta-data of a PR"""
- def setUp(self):
- """Set up the environnment, ran before every tests."""
- super(TestTicketAccessEditPRMetadata, self).setUp()
- tests.create_projects(self.session)
- tests.create_projects_git(
- os.path.join(self.path, "requests"), bare=True
- )
- set_up_git_repo(
- self.session, self.path, new_project=None, branch_from="feature"
- )
- # Add user "foo" to the project "test"
- repo = pagure.lib.query._get_project(self.session, "test")
- msg = pagure.lib.query.add_user_to_project(
- session=self.session,
- project=repo,
- new_user="foo",
- user="pingou",
- access="ticket",
- )
- self.session.commit()
- self.assertEqual(msg, "User added")
- def test_unauth_cannot_view_edit_metadata_ui(self):
- """Test that unauthenticated users cannot view the edit the
- metadata fields in the UI."""
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n"
- " - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-fw fa-pencil"></i></a>',
- '<a class="btn btn-outline-primary border-0 btn-sm '
- "issue-metadata-display editmetadatatoggle pointer inline-block"
- '"><i class="fa fa-fw fa-pencil"></i></a>',
- output_text,
- )
- self.assertNotIn(
- '<form method="POST" action="/test/pull-request/1/update">',
- output_text,
- )
- def test_admin_can_view_edit_metadata_ui(self):
- """Test that admin users can view the edit the metadata fields in
- the UI."""
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n"
- " - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-fw fa-pencil"></i></a>',
- '<a class="btn btn-outline-primary border-0 btn-sm '
- "issue-metadata-display editmetadatatoggle pointer inline-block"
- '"><i class="fa fa-fw fa-pencil"></i></a>',
- output_text,
- )
- self.assertIn(
- '<form method="POST" action="/test/pull-request/1/update">',
- output_text,
- )
- def test_admin_can_edit_metadata_ui(self):
- """Test that admin users can edit the metadata in the UI."""
- user = tests.FakeUser(username="pingou")
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": self.get_csrf(), "user": "foo"}
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n"
- " - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-fw fa-pencil"></i></a>',
- '<a class="btn btn-outline-primary border-0 btn-sm '
- "issue-metadata-display editmetadatatoggle pointer inline-block"
- '"><i class="fa fa-fw fa-pencil"></i></a>',
- output_text,
- )
- self.assertIn(
- '<form method="POST" action="/test/pull-request/1/update">',
- output_text,
- )
- self.assertIn(
- '<input value="foo"\n name="user" '
- 'id="assignee" placeholder="username" >',
- output_text,
- )
- def test_ticket_can_view_edit_metadata_ui(self):
- """Test that users with ticket access can view the edit the
- metadata fields in the UI."""
- user = tests.FakeUser(username="foo")
- with tests.user_set(self.app.application, user):
- output = self.app.get("/test/pull-request/1")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n"
- " - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-fw fa-pencil"></i></a>',
- '<a class="btn btn-outline-primary border-0 btn-sm '
- "issue-metadata-display editmetadatatoggle pointer inline-block"
- '"><i class="fa fa-fw fa-pencil"></i></a>',
- output_text,
- )
- self.assertIn(
- '<form method="POST" action="/test/pull-request/1/update">',
- output_text,
- )
- def test_ticket_can_edit_metadata_ui(self):
- """Test that users with ticket access can edit the metadata in the
- UI."""
- user = tests.FakeUser(username="foo")
- with tests.user_set(self.app.application, user):
- data = {"csrf_token": self.get_csrf(), "user": "pingou"}
- output = self.app.post(
- "/test/pull-request/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>PR#1: PR from the feature branch - test\n"
- " - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<i class="fa fa-fw fa-pencil"></i></a>',
- '<a class="btn btn-outline-primary border-0 btn-sm '
- "issue-metadata-display editmetadatatoggle pointer inline-block"
- '"><i class="fa fa-fw fa-pencil"></i></a>',
- output_text,
- )
- self.assertIn(
- '<form method="POST" action="/test/pull-request/1/update">',
- output_text,
- )
- self.assertIn(
- '<input value="pingou"\n name="user" '
- 'id="assignee" placeholder="username" >',
- output_text,
- )
- if __name__ == "__main__":
- unittest.main(verbosity=2)
|