May. 3rd, 2020

github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
[personal profile] github
Branch: refs/heads/master
Home: https://github.com/dreamwidth/dw-free
Commit: 1f1d9e54a78476f797951a016663a7b4ccae076d
https://github.com/dreamwidth/dw-free/commit/1f1d9e54a78476f797951a016663a7b4ccae076d
Author: Kareila <kareila@dreamwidth.org>
Date: 2020-05-03 (Sun, 03 May 2020)

Changed paths:
M bin/checkconfig.pl

Log Message:
-----------
improvements to bin/checkconfig.pl

This takes an existing stub of an idea -- making sure that etc/config-local.pl exists -- and expands it to include config.pl and config-private.pl, as well as the configs in t/ for testing. If the files exist, they are also evaluated to make sure they compile properly (no missing punctuation).

Having this would have saved me approximately two hours of pain yesterday.
github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
[personal profile] github
Branch: refs/heads/master
Home: https://github.com/dreamwidth/dw-free
Commit: 1f15f63db6b3c3f1e6398cc8405c3dbf7aba4ede
https://github.com/dreamwidth/dw-free/commit/1f15f63db6b3c3f1e6398cc8405c3dbf7aba4ede
Author: Jen <kareila@dreamwidth.org>
Date: 2020-05-03 (Sun, 03 May 2020)

Changed paths:
M cgi-bin/DW/Auth/TOTP.pm
M cgi-bin/LJ/Console/Command/ChangeJournalType.pm
M t/atom-post.t
M t/auth-password.t
M t/auth-totp.t
M t/console-changecommunityadmin.t
M t/console-changejournaltype.t
M t/console-reset.t
M t/proto-post-edit-roundtrip.t
M t/rename.t

Log Message:
-----------
[#2639] fix or disable broken tests and related functionality (#2642)

* fix syntax errors

* remove unneeded calls to old password method

t/console-changecommunityadmin.t: There was a test to make sure the password of a community was blank after transferring ownership, but communities haven't been allowed to have passwords in a very long time.

t/proto-post-edit-roundtrip.t: Use string variables and the check_password method instead.

t/console-reset.t: Use a known password and the check_password method instead.

* disable broken functionality for now

The change_journal_type console command won't work until we decide how to update the expected behavior, which has been to assign the password of the community's owner to the affected journal when changing it from a community to a personal journal. For now, the console command is disabled and all the relevant tests are skipped.

As for the t/atom-post.t file, it seems linked to AtomAPI authentication, which is a rarely used feature of the site. Skipping these tests for now but not sure if updating or removing is the best path forward.

* remove password-matching test from t/rename.t

Since it is no longer possible for us to test that two users have the same password, we no longer enforce this as a constraint when deciding if a rename is allowed. Now we only make sure that the authenticated email addresses on the two accounts match.

* tidy

Profile

Dreamwidth Changelog

August 2025

S M T W T F S
     12
3456789
10111213141516
171819202122 23
2425262728 29 30
31      

Most Popular Tags

Style Credit

Expand Cut Tags

No cut tags
Page generated Sep. 17th, 2025 10:11 pm
Powered by Dreamwidth Studios