Skip to content

[No QA] Add test for matching Translations keys #4090

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jul 29, 2021
Merged

[No QA] Add test for matching Translations keys #4090

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f31e802
add test for Translations keys match
parasharrajat Jul 15, 2021
9e0fe2a
added translations
parasharrajat Jul 17, 2021
6ad65b7
fix: translation tests
parasharrajat Jul 29, 2021
26626ec
fix translation files
parasharrajat Jul 29, 2021
b1e7dc2
Pull latest changes
parasharrajat Jul 29, 2021
5551926
added missing key
parasharrajat Jul 29, 2021
158dfb3
refactor tests
parasharrajat Jul 29, 2021
72a3280
pulled latest changes and Show test failure
parasharrajat Jul 29, 2021
f9baac6
fix: tests
parasharrajat Jul 29, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
133 changes: 94 additions & 39 deletions src/languages/es.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ export default {
cancel: 'Cancelar',
yes: 'Si',
no: 'No',
ok: 'OK',
attachment: 'Archivo Adjunto',
to: 'A',
optional: 'Opcional',
Expand All @@ -21,10 +22,14 @@ export default {
not: 'No',
signIn: 'Conectarse',
continue: 'Continuar',
firstName: 'Primer nombre',
lastName: 'Apellido',
phoneNumber: 'Número de teléfono',
email: 'Email',
and: 'y',
details: 'Detalles',
privacy: 'Intimidad',
privacyPolicy: 'Política de privacidad',
delete: 'Eliminar',
deleted: 'eliminado',
contacts: 'Contactos',
Expand Down Expand Up @@ -105,11 +110,14 @@ export default {
writeSomething: 'Escribe algo...',
blockedFromConcierge: 'Comunicación no permitida',
youAppearToBeOffline: 'Parece que estás desconectado.',
fileUploadFailed: 'Subida fallida. El archivo no es compatible.',
roomIsArchived: 'Esta sala de chat ha sido eliminada',
},
reportActionContextMenu: {
contextMenuItem: {
copyToClipboard: 'Copiar al Portapapeles',
copied: '¡Copiado!',
},
reportActionContextMenu: {
copyLink: 'Copiar Enlace',
markAsUnread: 'Marcar como no leído',
editComment: 'Editar Commentario',
Expand Down Expand Up @@ -138,6 +146,7 @@ export default {
confirm: 'Confirmar',
splitBill: 'Dividir Factura',
requestMoney: 'Pedir Dinero',
sendMoney: 'Enviar Dinero',
pay: 'Pagar',
viewDetails: 'Ver detalles',
settleExpensify: 'Pagar con Expensify',
Expand All @@ -149,6 +158,7 @@ export default {
owes: ({manager, owner}) => `${manager} debe a ${owner}`,
paid: ({owner, manager}) => `${manager} pagó a ${owner}`,
split: ({amount}) => `Dividir ${amount}`,
send: ({amount}) => `Enviar ${amount}`,
choosePaymentMethod: 'Elige el método de pago:',
noReimbursableExpenses: 'El monto de este informe es inválido',
},
Expand All @@ -172,9 +182,7 @@ export default {
profilePage: {
profile: 'Perfil',
tellUsAboutYourself: '¡Cuéntanos algo sobre tí, nos encantaría conocerte!',
firstName: 'Nombre',
john: 'Juan',
lastName: 'Apellidos',
doe: 'Nadie',
preferredPronouns: 'Pronombres preferidos',
selectYourPronouns: 'Selecciona tus pronombres',
Expand Down Expand Up @@ -210,6 +218,12 @@ export default {
desktop: {
label: 'Desktop',
},
window: {
label: 'Desktop',
},
Rajat: {
label: 'Desktop',
},
},
signOut: 'Desconectar',
versionLetter: 'v',
Expand All @@ -233,7 +247,8 @@ export default {
enterYourUsernameToGetPaidViaPayPal: 'Escribe tu nombre de usuario para que otros puedan pagarte a través de PayPal.',
payPalMe: 'PayPal.me/',
yourPayPalUsername: 'Tu usuario de PayPal',
addPayPalAccount: 'Agregar cuenta de PayPal',
addPayPalAccount: 'Agregar Cuenta de PayPal',
growlMessageOnSave: 'Su nombre de usuario de PayPal se agregó correctamente',
editPayPalAccount: 'Actualizar cuenta de PayPal',
},
paymentsPage: {
Expand All @@ -246,7 +261,6 @@ export default {
addFirstPaymentMethod: 'Añade un método de pago para enviar y recibir pagos directamente desde la aplicación',
},
preferencesPage: {
mostRecent: 'Más Recientes',
mostRecentModeDescription: 'Esta opción muestra por defecto todos los chats, ordenados a partir del más reciente, con los chats destacados arriba de todo',
focus: '#concentración',
focusModeDescription: '#concentración – Muestra sólo los chats no leídos y destacados ordenados alfabéticamente.',
Expand All @@ -263,6 +277,14 @@ export default {
expensifyIsOpenSource: 'Expensify.cash es open source',
theCode: 'el código',
openJobs: 'trabajos disponibles',
heroHeading: 'Dividir cuentas\ny chatear con amigos.',
heroDescription: {
phrase1: 'El dinero habla. Y ahora que el chat y los pagos están en un solo lugar, también es fácil. Sus pagos le llegan tan rápido como puede transmitir su punto.',
phrase2: 'New Expensify es de código abierto. Vista',
phrase3: 'el código',
phrase4: 'Vista',
phrase5: 'vacantes',
},
},
termsOfUse: {
phrase1: 'Al usar Expensify.cash, estás aceptando los',
Expand All @@ -271,14 +293,15 @@ export default {
phrase4: 'política de privacidad',
phrase5: 'El envío de dinero es brindado por Expensify Payments LLC (NMLS ID:2017010) de conformidad con sus',
phrase6: 'licencias',
phrase7: 'licenses',
},
passwordForm: {
pleaseFillOutAllFields: 'Por favor completa todos los campos',
enterYourTwoFactorAuthenticationCodeToContinue: 'Ingrese su código de autenticación de dos factores para continuar',
forgot: '¿Te has olvidado?',
twoFactorCode: 'Autenticación de 2 factores',
requiredWhen2FAEnabled: 'Obligatorio cuando A2F está habilitado',
error: {
incorrectLoginOrPassword: 'Usuario o clave incorrectos. Por favor inténtalo de nuevo',
twoFactorAuthenticationEnabled: 'Tienes autenticación de 2 factores activada en esta cuenta. Por favor conéctate usando su email o número de teléfono',
invalidLoginOrPassword: 'Usuario o clave incorrectos. Por favor inténtalo de nuevo o resetea tu clave',
unableToResetPassword: 'No pudimos cambiar tu clave. Probablemente porque el enlace para resetear la clave ha expirado. Te hemos enviado un nuevo enlace. Chequea tu bandeja de entrada y tu carpeta de Spam',
Expand Down Expand Up @@ -309,7 +332,10 @@ export default {
},
setPasswordPage: {
enterPassword: 'Escribe una contraseña',
confirmNewPassword: 'Confirma la contraseña',
setPassword: 'Configura tu Contraseña',
passwordsDontMatch: 'Las contraseñas deben coincidir',
newPasswordPrompt: 'Su contraseña debe tener al menos 8 caracteres, \n1 letra mayúscula, 1 letra minúscula, 1 número.',
},
bankAccount: {
accountNumber: 'Número de cuenta',
Expand Down Expand Up @@ -372,6 +398,68 @@ export default {
noPhoneNumber: 'Por favor escribe un número de teléfono que incluya el código de país e.g +447814266907',
maxParticipantsReached: 'Has llegado al número máximo de participantes para un grupo.',
},
onfidoStep: {
acceptTerms: 'Al continuar con la solicitud para activar su billetera Expensify, confirma que ha leído, comprende y acepta ',
facialScan: 'Política y lanzamiento de la exploración facial de Onfido',
tryAgain: 'Intentar otra vez',
verifyIdentity: 'Verificar identidad',
genericError: 'Hubo un error al procesar este paso. Inténtalo de nuevo.',
},
additionalDetailsStep: {
headerTitle: 'Detalles adicionales',
helpText: 'Necesitamos confirmar la siguiente información antes de que podamos procesar este pago.',
helpLink: 'Obtenga más información sobre por qué necesitamos esto.',
legalFirstNameLabel: 'Primer nombre legal',
legalMiddleNameLabel: 'Segundo nombre legal',
legalLastNameLabel: 'Apellido legal',
},
termsStep: {
headerTitle: 'Condiciones y tarifas',
haveReadAndAgree: 'He leído y acepto recibir ',
electronicDisclosures: 'divulgaciones electrónicas',
agreeToThe: 'Estoy de acuerdo con la ',
walletAgreement: 'Acuerdo de billetera',
enablePayments: 'Habilitar pagos',
termsMustBeAccepted: 'Se deben aceptar los términos',
},
activateStep: {
headerTitle: 'Habilitar pagos',
activated: 'Su billetera Expensify está lista para usar.',
checkBackLater: 'Todavía estamos revisando tu información. Por favor, vuelva más tarde.',
},
companyStep: {
headerTitle: 'Información de la Empresa',
subtitle: 'Dé más información sobre su empresa.',
legalBusinessName: 'Nombre Comercial Legal',
companyWebsite: 'Company Website',
taxIDNumber: 'Tax ID Number',
companyType: 'Página Web de la Empresa',
incorporationDate: 'Fecha de Incorporación',
industryClassificationCode: 'Código de Clasificación Industrial',
confirmCompanyIsNot: 'Confirmo que esta empresa no está en el',
listOfRestrictedBusinesses: 'lista de negocios restringidos',
incorporationDatePlaceholder: 'Fecha de inicio (aaaa-mm-dd)',
companyPhonePlaceholder: '10 dígitos, sin guiones',
},
requestorStep: {
headerTitle: 'Información del solicitante',
financialRegulations: 'Las leyes fiscales y el reglamento bancario nos obliga a verificar la identidad de todo individuo que desee añadir una cuenta bancaria representando a una compañía. ',
learnMore: 'Más información',
isMyDataSafe: '¿Están seguros mis datos?',
onFidoConditions: 'Al continuar con la solicitud de añadir esta cuenta bancaria, confirma que ha leído, entiende y acepta ',
onFidoFacialScan: 'Onfido’s Facial Scan Policy and Release',
isControllingOfficer: 'Estoy autorizado a utilizar la cuenta bancaria de mi compañía para gastos de empresa',
isControllingOfficerError: 'Debe ser un oficial controlador con autorización para operar la cuenta bancaria de la compañía',
},
validationStep: {
headerTitle: 'Validar',
buttonText: 'Finalizar Configuración',
maxAttemptError: 'Se ha inhabilitado la validación de esta cuenta bancaria, debido a demasiados intentos incorrectos. Por favor contáctenos.',
description: 'Uno o dos días después de agregar su cuenta a Expensify, enviamos tres (3) transacciones a su cuenta. Tienen una línea comercial como "Expensify, Inc. Validation"',
descriptionCTA: 'Ingrese el monto de cada transacción en los campos a continuación. Ejemplo: 1.51',
reviewingInfo: '¡Gracias! Estamos revisando tu información y nos comunicaremos contigo en breve. Consulte su chat con Concierge ',
forNextSteps: ' para conocer los próximos pasos para terminar de configurar su cuenta bancaria.',
},
beneficialOwnersStep: {
beneficialOwners: 'Beneficial Owners',
additionalInformation: 'Additional Information',
Expand Down Expand Up @@ -433,39 +521,6 @@ export default {
welcomeNote: ({workspaceName}) => `¡Has sido invitado a la ${workspaceName} Espacio de trabajo! Descargue la aplicación móvil Expensify para comenzar a rastrear sus gastos.`,
},
},
companyStep: {
headerTitle: 'Información de la Empresa',
subtitle: 'Dé más información sobre su empresa.',
legalBusinessName: 'Nombre Comercial Legal',
companyWebsite: 'Company Website',
taxIDNumber: 'Tax ID Number',
companyType: 'Página Web de la Empresa',
incorporationDate: 'Fecha de Incorporación',
industryClassificationCode: 'Código de Clasificación Industrial',
confirmCompanyIsNot: 'Confirmo que esta empresa no está en el',
listOfRestrictedBusinesses: 'lista de negocios restringidos',
incorporationDatePlaceholder: 'Fecha de inicio (aaaa-mm-dd)',
companyPhonePlaceholder: '10 dígitos, sin guiones',
},
validationStep: {
headerTitle: 'Validar',
buttonText: 'Finalizar Configuración',
maxAttemptError: 'Se ha inhabilitado la validación de esta cuenta bancaria, debido a demasiados intentos incorrectos. Por favor contáctenos.',
description: 'Uno o dos días después de agregar su cuenta a Expensify, enviamos tres (3) transacciones a su cuenta. Tienen una línea comercial como "Expensify, Inc. Validation"',
descriptionCTA: 'Ingrese el monto de cada transacción en los campos a continuación. Ejemplo: 1.51',
reviewingInfo: '¡Gracias! Estamos revisando tu información y nos comunicaremos contigo en breve. Consulte su chat con Concierge ',
forNextSteps: ' para conocer los próximos pasos para terminar de configurar su cuenta bancaria.',
},
requestorStep: {
headerTitle: 'Información del solicitante',
financialRegulations: 'Las leyes fiscales y el reglamento bancario nos obliga a verificar la identidad de todo individuo que desee añadir una cuenta bancaria representando a una compañía. ',
learnMore: 'Más información',
isMyDataSafe: '¿Están seguros mis datos?',
onFidoConditions: 'Al continuar con la solicitud de añadir esta cuenta bancaria, confirma que ha leído, entiende y acepta ',
facialScan: 'la política de reconocimiento facial y la exención de Onfido',
isControllingOfficer: 'Estoy autorizado a utilizar la cuenta bancaria de mi compañía para gastos de empresa',
isControllingOfficerError: 'Debe ser un oficial controlador con autorización para operar la cuenta bancaria de la compañía',
},
requestCallPage: {
requestACall: 'Llámame por teléfono',
description: '¿Necesitas ayuda configurando tu cuenta? Nuestro equipo de guías puede ayudarte.',
Expand Down
43 changes: 42 additions & 1 deletion tests/unit/TranslateTest.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
const _ = require('underscore');
const translate = require('../../src/libs/translate');
const translations = require('../../src/languages/translations');
const CONFIG = require('../../src/CONFIG');
const translations = require('../../src/languages/translations');

const originalTranslations = _.clone(translations);
translations.default = {
en: {
testKey1: 'English',
Expand Down Expand Up @@ -52,3 +54,42 @@ describe('translate', () => {
expect(translate.translate('en', ['testKeyGroup', 'testFunction'], {testVariable})).toBe(expectedValue);
});
});

describe('Translation Keys', () => {
function traverseKeyPath(source, path, keyPaths) {
const pathArray = keyPaths || [];
const keyPath = path ? `${path}.` : '';
_.each(_.keys(source), (key) => {
if (_.isObject(source[key]) && !_.isFunction(source[key])) {
traverseKeyPath(source[key], keyPath + key, pathArray);
} else {
pathArray.push(keyPath + key);
}
});
return pathArray;
}
const excludeLanguages = ['en', 'es-ES'];
const languages = _.without(_.keys(originalTranslations.default), ...excludeLanguages);
const mainLanguage = originalTranslations.default.en;
const mainLanguageKeys = traverseKeyPath(mainLanguage);

_.each(languages, (ln) => {
const languageKeys = traverseKeyPath(originalTranslations.default[ln]);

it(`Does ${ln} locale has all the keys`, () => {
const hasAllKeys = _.difference(mainLanguageKeys, languageKeys);
if (hasAllKeys.length) {
console.debug(`🏹 [ ${hasAllKeys.join(', ')} ] are missing from ${ln}.js`);
}
expect(hasAllKeys.length).toBe(0);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, I was thinking, maybe we could do this and get a better error message?

Suggested change
expect(hasAllKeys.length).toBe(0);
expect(hasAllKeys).toBeEqual([]);

I think this will show in the test result exactly which keys are missing, no?
(same applies to the one below)

Copy link
Member Author

@parasharrajat parasharrajat Jul 29, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does not show the keys that are not correct. It just shows Expected and produced output. So all the correct and wrong keys would be shown as a big string which is like 50 lines long.

 Expected: 0 <-- would be all the keys on main language in green color.
 Received: 2 <-- would be all the keys on matching language in red color.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested with expect.objectContaining and expect.arrayContaining.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmmm, can we change the code to do _.without(mainLagunageKeys, ...languageKeys)? That would return only the missing keys, right?

Copy link
Member Author

@parasharrajat parasharrajat Jul 29, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It currently only gives the missing keys. Again _.without(mainLagunageKeys, ...languageKeys) will go in console.log. @iwiznia

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The console log is not good enough as it is super hard to find the right message. In any case I don't think you understood what I meant, because I just tried and it works. Here's the test code:

        it(`Does ${ln} locale has all the keys`, () => {
            const hasAllKeys = _.without(mainLanguageKeys, ...languageKeys);
            expect(hasAllKeys).toEqual([]);
        });

        it(`Does ${ln} locale has unused keys`, () => {
            const hasAllKeys = _.without(languageKeys, ...mainLanguageKeys);
            expect(hasAllKeys).toEqual([]);
        });

And this is the output:

 ● Translation Keys › Does es locale has all the keys

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 4

    - Array []
    + Array [
    +   "preferencesPage.mostRecent",
    +   "passwordForm.error.incorrectLoginOrPassword",
    + ]

      79 |         it(`Does ${ln} locale has all the keys`, () => {
      80 |             const hasAllKeys = _.without(mainLanguageKeys, ...languageKeys);
    > 81 |             expect(hasAllKeys).toEqual([]);
         |                                ^
      82 |         });
      83 |
      84 |         it(`Does ${ln} locale has unused keys`, () => {

      at Object.<anonymous> (tests/unit/TranslateTest.js:81:32)

  ● Translation Keys › Does es locale has unused keys

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 4

    - Array []
    + Array [
    +   "initialSettingsPage.appDownloadLinks.window.label",
    +   "initialSettingsPage.appDownloadLinks.Rajat.label",
    + ]

      84 |         it(`Does ${ln} locale has unused keys`, () => {
      85 |             const hasAllKeys = _.without(languageKeys, ...mainLanguageKeys);
    > 86 |             expect(hasAllKeys).toEqual([]);
         |                                ^
      87 |         });
      88 |     });
      89 | });

      at Object.<anonymous> (tests/unit/TranslateTest.js:86:32)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh Yeah. This way I got it now.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@iwiznia Check this out https://github.com/Expensify/App/actions/runs/1079637864. I added annotations.
image

});

it(`Does ${ln} locale has unused keys`, () => {
const hasAllKeys = _.difference(languageKeys, mainLanguageKeys);
if (hasAllKeys.length) {
console.debug(`🏹 [ ${hasAllKeys.join(', ')} ] are unused keys in ${ln}.js`);
}
expect(hasAllKeys.length).toBe(0);
});
});
});