Page MenuHomePhorge

Update PassphraseCredential to support Mentions
ClosedPublic

Authored by Ekubischta on Jan 21 2022, 20:42.
Tags
None
Referenced Files
F2157697: D25032.diff
Wed, Apr 24, 22:54
Unknown Object (File)
Mon, Apr 22, 21:45
Unknown Object (File)
Sun, Apr 21, 20:49
Unknown Object (File)
Mon, Apr 8, 10:32
Unknown Object (File)
Mon, Apr 8, 10:24
Unknown Object (File)
Mon, Apr 8, 10:11
Unknown Object (File)
Sun, Apr 7, 10:23
Unknown Object (File)
Mon, Apr 1, 00: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