Karavel/fixin.me
0
0
Fork 0
forked from fixin.me/fixin.me
Code Pull Requests Activity

Merging from main master to my repo master. #4

Closed
Karavel wants to merge 53 commits from fixin.me/fixin.me:master into master
pull from: fixin.me/fixin.me:master
merge into: Karavel:master
Conversation 0 Commits 53 Files Changed 50 +4 -4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit dcffa86e93 - Show all commits

8
test/system/units_test.rb
View File

@ -23,7 +23,7 @@ class UnitsTest < ApplicationSystemTestCase
end
end
test "add unit" do
test "new" do
add_link = all(:link, exact_text: ADD_UNIT_LABELS.sample).sample
add_link.click
assert_equal 'disabled', add_link[:disabled]
@ -55,12 +55,12 @@ class UnitsTest < ApplicationSystemTestCase
assert_selector '.flash.notice', text: t('units.create.success', unit: @user.units.last.symbol)
end
# TODO: check proper form/button redisplay and flash messages
test "add unit on validation error" do
# TODO: check proper form/button redisplay and flash messages on add/edit
test "new and edit form on validation error" do
end
# TODO: add non-empty form closing warning
test "add and edit disallow opening multiple forms" do
test "new and edit disallow opening multiple forms" do
# Once new/edit form is open, attempt to open another one will close it
links = {}
targets = {}