diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-16 13:50:15 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-16 13:50:15 +0200 |
| commit | 603ccfbd5fe96f0b83aaf3d5118aded2a39992fb (patch) | |
| tree | 0e88ec41af6ce1a03e65bedb3f68a5a9881d96bb /test/models/node_action_test.rb | |
| parent | bb7ef80d084f474bb4c3be0ae0d033aff3de0272 (diff) | |
Condense the action log into a scannable table
Several minor improvements to the action log presentation:
* Now a table with date and human readable presentation as
rows is displayed
* If no changes in a title were detected, the old version
is omitted
* The "inferred" flag is demoted to the end of the line
* You can zoom in on the node's history directly from a
log line
* byline for the first publish action is preserved
* Revisions are directly linked to when a new one i
published
Diffstat (limited to 'test/models/node_action_test.rb')
| -rw-r--r-- | test/models/node_action_test.rb | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/test/models/node_action_test.rb b/test/models/node_action_test.rb index 24d8245..b177cca 100644 --- a/test/models/node_action_test.rb +++ b/test/models/node_action_test.rb | |||
| @@ -18,6 +18,12 @@ class NodeActionTest < ActiveSupport::TestCase | |||
| 18 | assert_equal({ :title => { "from" => nil, "to" => "Erstausgabe" } }, diff) | 18 | assert_equal({ :title => { "from" => nil, "to" => "Erstausgabe" } }, diff) |
| 19 | end | 19 | end |
| 20 | 20 | ||
| 21 | test "first publish carries the byline when the page has one" do | ||
| 22 | diff = NodeAction.head_diff(nil, build_page(:user => users(:quentin))) | ||
| 23 | |||
| 24 | assert_equal({ "from" => nil, "to" => users(:quentin).login }, diff[:author]) | ||
| 25 | end | ||
| 26 | |||
| 21 | test "title pair is always present, even when unchanged" do | 27 | test "title pair is always present, even when unchanged" do |
| 22 | diff = NodeAction.head_diff(build_page, build_page) | 28 | diff = NodeAction.head_diff(build_page, build_page) |
| 23 | 29 | ||
| @@ -109,4 +115,14 @@ class NodeActionTest < ActiveSupport::TestCase | |||
| 109 | 115 | ||
| 110 | assert_nil NodeAction.head_diff(old_page, new_page)[:translation_diff] | 116 | assert_nil NodeAction.head_diff(old_page, new_page)[:translation_diff] |
| 111 | end | 117 | end |
| 118 | |||
| 119 | test "record! passes provenance and historical timestamps through" do | ||
| 120 | long_ago = 2.years.ago | ||
| 121 | action = NodeAction.record!(:node => nil, :action => "publish", | ||
| 122 | :occurred_at => long_ago, | ||
| 123 | :inferred_from => "from_page_revision") | ||
| 124 | |||
| 125 | assert_equal "from_page_revision", action.inferred_from | ||
| 126 | assert_in_delta long_ago.to_f, action.occurred_at.to_f, 1.0 | ||
| 127 | end | ||
| 112 | end | 128 | end |
