Page MenuHomePhorge

Update PassphraseCredential to support Mentions
ClosedPublic

Authored by Ekubischta on Jan 21 2022, 20:42.
Tags
None
Referenced Files
F3306513: D25032.1743129283.diff
Thu, Mar 27, 02:34
F3297469: D25032.1742985678.diff
Tue, Mar 25, 10:41
F3296663: D25032.1742976129.diff
Tue, Mar 25, 08:02
F3289689: D25032.1742851642.diff
Sun, Mar 23, 21:27
F3289402: D25032.1742846265.diff
Sun, Mar 23, 19:57
F3282341: D25032.1742713567.diff
Sat, Mar 22, 07:06
F3225163: D25032.1742094168.diff
Sat, Mar 15, 03:02
F3224802: D25032.1742089222.diff
Sat, Mar 15, 01:40

Details

Summary

This revision allows us to see where passphrase credentials are mentioned in other parts of phabricator

This is very useful to "What is this credential for?" If we can see where it was been mentioned

Test Plan

Mentioned a passphrase and saw it in the transaction list

Diff Detail

Repository
rP Phorge
Branch
lc-phorge-master
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 60
Build 60: arc lint + arc unit

Unit TestsFailed

TimeTest
967 msPhabricatorLibraryTestCase::testLibraryMap
Assertion failed, expected 'true' (at PhutilLibraryTestCase.php:51): The library map is out of date. Rebuild it with `arc liberate`. These entries differ: xmap.PassphraseCredential.4, xmap.PassphraseCredential.5, xmap.PassphraseCredential.6, xmap.PassphraseCredential.7, xmap.PassphraseCredential.8, xmap.PassphraseCredential.9.
949 msPhabricatorCelerityTestCase::testCelerityMaps
1 assertion passed.
1,571 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
316 msPhabricatorInfrastructureTestCase::testApplicationsInstalled
1 assertion passed.
345 msPhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries
1 assertion passed.
View Full Test Results (1 Failed · 6 Passed)

Event Timeline

Ekubischta updated this revision to Diff 85.
  • ran arc liberate
This revision is now accepted and ready to land.Oct 4 2022, 19:04