Fix false positive in Catalan 'noi' filter (#252)

Author: anortef

inclusion/language.go

@@ -69,7 +69,7 @@ var inclusiveFilters = []InclusiveFilter{
 	{Filter: "retras mental", Reply: "*Retrassat/retrassada* i *Retràs mental* són termes despectius eliminats del vocabulari psiquiàtric i, segons l'OMS, la forma correcta per referir-vos a aquest grup de malalties i trastorns és *Trastorn del desenvolupament intel·lectual*"},
 	{Filter: "retarda(t|da)", Reply: "*Retardat/retardada* i *Retard mental* són termes despectius eliminats del vocabulari psiquiàtric i, segons l'OMS, la forma correcta per referir-vos a aquest grup de malalties i trastorns és *Trastorn del desenvolupament intel·lectual*"},
 	{Filter: "retard mental", Reply: "*Retardat/retardada* i *Retard mental* són termes despectius eliminats del vocabulari psiquiàtric i, segons l'OMS, la forma correcta per referir-vos a aquest grup de malalties i trastorns és *Trastorn del desenvolupament intel·lectual*"},
-	{Filter: "noi(a|es|s)*", Reply: "En lloc de *noi(a|es|s)*, potser et refereixes a *gent*?... *[Si us plau, considera editar el teu missatge perquè sigui més inclusiu]*"},
+	{Filter: "noi(a|es|s)?", Reply: "En lloc de *noi(a|es|s)*, potser et refereixes a *gent*?... *[Si us plau, considera editar el teu missatge perquè sigui més inclusiu]*"},
 	{Filter: "bo(g|j)eria", Reply: "La paraula *bogeria* és considerada per algunes persones com a irrespectuosa cap a les persones que pateixen alguna malaltia mental.\nPotser has volgut dir *indignant*, *impensable*, *absurd* o *incomprensible*? Has considerat fer servir un adjectiu diferent com *ridícul*?"},
 	{Filter: "bo(ig|ja|jos)", Reply: "La paraula *boig/boja* és considerada per algunes persones com a irrespectuosa cap a les persones que pateixen alguna malaltia mental.\nPotser has volgut dir *indignant*, *impensable*, *absurd* o *incomprensible*? Has considerat fer servir un adjectiu diferent com *ridícul*?"},
 }

inclusion/language_test.go

@@ -23,7 +23,14 @@ func TestFilter(t *testing.T) {
 		{name: "'ladies' is not usually used in the right context", input: "hi ladies!", filtered: true},
 		{name: "'retrassada' should be 'Trastorn del desenvolupament intel·lectual'", input: "L'entrega ha sigut retrassada", filtered: true},
 		{name: "'noi' should be 'persona'", input: "És un noi simpàtic", filtered: true},
+		{name: "'noies' should be 'gent'", input: "Hola noies", filtered: true},
+		{name: "'nois' should be 'gent'", input: "Hola nois", filtered: true},
+		{name: "'noia' should be 'persona'", input: "És una noia simpàtica", filtered: true},
 		{name: "'noible' should be ok", input: "És una persona noible", filtered: false},
+		{name: "URL containing 'noises' should be ok", input: "https://abcnews.com/US/meow-meow-pilots-scolded-after-animal-noises-heard/story?id=132076661", filtered: false},
+		{name: "'noises' inside a sentence should be ok", input: "the animal noises were loud", filtered: false},
+		{name: "'paranoid' should be ok", input: "don't be paranoid", filtered: false},
+		{name: "'noir' should be ok", input: "I love film noir", filtered: false},
 	}
 
 	extraFilters := []InclusiveFilter{