Change:
assert_eq!(LangString::parse(s, ErrorCodes::Yes, true, None), lg) } <del> t(LangString::all_false()); t(LangString { original: "rust".into(), ..LangString::all_false() }); t(LangString { original: "sh".into(), rust: false, ..LangString::all_false() }); t(LangString { original: "ignore".into(), ignore: Ignore::All, ..LangString::all_false() }); </del> <ins> t(Default::default()); t(LangString { original: "rust".into(), ..Default::default() }); t(LangString { original: "sh".into(), rust: false, ..Default::default() }); t(LangString { original: "ignore".into(), ignore: Ignore::All, ..Default::default() }); </ins> t(LangString { original: "ignore-foo".into(), ignore: Ignore::Some(vec!["foo".to_string()]), <del> ..LangString::all_false() }); t(LangString { original: "should_panic".into(), should_panic: true, ..LangString::all_false() }); t(LangString { original: "no_run".into(), no_run: true, ..LangString::all_false() }); t(LangString { original: "test_harness".into(), test_harness: true, ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); <ins> t(LangString { original: "should_panic".into(), should_panic: true, ..Default::default() }); t(LangString { original: "no_run".into(), no_run: true, ..Default::default() }); t(LangString { original: "test_harness".into(), test_harness: true, ..Default::default() }); </ins> t(LangString { original: "compile_fail".into(), no_run: true, compile_fail: true, <del> ..LangString::all_false() }); t(LangString { original: "allow_fail".into(), allow_fail: true, ..LangString::all_false() }); t(LangString { original: "{.no_run .example}".into(), no_run: true, ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); <ins> t(LangString { original: "allow_fail".into(), allow_fail: true, ..Default::default() }); t(LangString { original: "{.no_run .example}".into(), no_run: true, ..Default::default() }); </ins> t(LangString { original: "{.sh .should_panic}".into(), should_panic: true, rust: false, <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); <del> t(LangString { original: "{.example .rust}".into(), ..LangString::all_false() }); </del> <ins> t(LangString { original: "{.example .rust}".into(), ..Default::default() }); </ins> t(LangString { original: "{.test_harness .rust}".into(), test_harness: true, <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); t(LangString { original: "text, no_run".into(), no_run: true, rust: false, <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); t(LangString { original: "text,no_run".into(), no_run: true, rust: false, <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); t(LangString { original: "edition2015".into(), edition: Some(Edition::Edition2015), <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); t(LangString { original: "edition2018".into(), edition: Some(Edition::Edition2018), <del> ..LangString::all_false() </del> <ins> ..Default::default() </ins> }); }

Discussion:
is not actually allfalse - will return . It would be nice to rename it to at some point (and maybe add while we're at it). cc Originally posted by in <!-- TRIAGEBOTSTART --<!-- TRIAGEBOTASSIGNSTART --<!-- TRIAGEBOTASSIGNDATASTARTTRIAGEBOTASSIGNDATAEND --<!-- TRIAGEBOTASSIGNEND --<!-- TRIAGEBOTEND --
claim I'll either get to it before next week is over or when is merged, whichever comes first