mirror of
https://github.com/xmonad/xmonad.git
synced 2025-07-31 12:11:53 -07:00
incorrect invariant test for greedyView
This commit is contained in:
@@ -139,7 +139,7 @@ prop_view_I (n :: NonNegative Int) (x :: T) =
|
|||||||
n `tagMember` x ==> invariant $ view (fromIntegral n) x
|
n `tagMember` x ==> invariant $ view (fromIntegral n) x
|
||||||
|
|
||||||
prop_greedyView_I (n :: NonNegative Int) (x :: T) =
|
prop_greedyView_I (n :: NonNegative Int) (x :: T) =
|
||||||
n `tagMember` x ==> invariant $ view (fromIntegral n) x
|
n `tagMember` x ==> invariant $ greedyView (fromIntegral n) x
|
||||||
|
|
||||||
prop_focusUp_I (n :: NonNegative Int) (x :: T) =
|
prop_focusUp_I (n :: NonNegative Int) (x :: T) =
|
||||||
invariant $ foldr (const focusUp) x [1..n]
|
invariant $ foldr (const focusUp) x [1..n]
|
||||||
@@ -236,6 +236,13 @@ prop_greedyView_current (x :: T) (n :: NonNegative Int) = i `tagMember` x ==>
|
|||||||
where
|
where
|
||||||
i = fromIntegral n
|
i = fromIntegral n
|
||||||
|
|
||||||
|
-- greedyView leaves things unchanged for invalid workspaces
|
||||||
|
prop_greedyView_current_id (x :: T) (n :: NonNegative Int) = not (i `tagMember` x) ==>
|
||||||
|
tag (workspace $ current (greedyView i x)) == j
|
||||||
|
where
|
||||||
|
i = fromIntegral n
|
||||||
|
j = tag (workspace (current x))
|
||||||
|
|
||||||
-- greedyView *only* sets the current workspace, and touches Xinerama.
|
-- greedyView *only* sets the current workspace, and touches Xinerama.
|
||||||
-- no workspace contents will be changed.
|
-- no workspace contents will be changed.
|
||||||
prop_greedyView_local (x :: T) (n :: NonNegative Int) = i `tagMember` x ==>
|
prop_greedyView_local (x :: T) (n :: NonNegative Int) = i `tagMember` x ==>
|
||||||
@@ -675,6 +682,7 @@ main = do
|
|||||||
|
|
||||||
,("greedyView : invariant" , mytest prop_greedyView_I)
|
,("greedyView : invariant" , mytest prop_greedyView_I)
|
||||||
,("greedyView sets current" , mytest prop_greedyView_current)
|
,("greedyView sets current" , mytest prop_greedyView_current)
|
||||||
|
,("greedyView is safe " , mytest prop_greedyView_current_id)
|
||||||
,("greedyView idempotent" , mytest prop_greedyView_idem)
|
,("greedyView idempotent" , mytest prop_greedyView_idem)
|
||||||
,("greedyView reversible" , mytest prop_greedyView_reversible)
|
,("greedyView reversible" , mytest prop_greedyView_reversible)
|
||||||
,("greedyView is local" , mytest prop_greedyView_local)
|
,("greedyView is local" , mytest prop_greedyView_local)
|
||||||
|
Reference in New Issue
Block a user