Merge pull request #64 from SensCritique/fix/new-extension-name

Release 1.2.15

CONTRIB.md

@@ -9,7 +9,7 @@ All the code is the same for Chrome and Firefox except for the event system whic
 - SensCritique (HTTP): Fetch video ID and Type
 - SensCritique (HTTP): Fetch video note
 ##### Content_script
-- Noteflix: Generate DOM with providers notes.
+- SensCritique-extension: Generate DOM with providers notes.
 
 # Installation / Test
 ## Installation

README.md

@@ -1,6 +1,6 @@
 <img src="/images/logo-128.png" height=50/>
 
-# NoteFlix
+# SensCritique-extension
 Permet d'avoir un score SensCritique sur les pages de descriptions Netflix
 
 [Cliquez ici pour installer l'extension depuis le store firefox](https://addons.mozilla.org/fr/firefox/addon/noteflix/)

__tests__/dom/Ratings.test.ts

@@ -6,5 +6,5 @@ test('It should render one div for SensCritique', () => {
 
   expect(ratingsDiv).not.toBeNull()
   expect(ratingsDiv.childNodes.length).toBe(1)
-  expect((ratingsDiv.childNodes[0] as Element).classList).toContain('senscritique_XXX')
+  expect((ratingsDiv.childNodes[0] as Element).classList).toContain('senscritique_extension_XXX')
 })

__tests__/dom/SensCritiqueRating.test.ts

@@ -13,9 +13,9 @@ test.each(successDataset)('It should convert SensCritique note (%s) in percentag
     name: 'test',
     redirect: ''
   })
-  const noteflixScore = sensCritiqueRating.ratingInPercent(sensCritiqueNote)
+  const sensCritiqueExtensionScore = sensCritiqueRating.ratingInPercent(sensCritiqueNote)
 
-  expect(noteflixScore).toBe(expected)
+  expect(sensCritiqueExtensionScore).toBe(expected)
 })
 
 const errorDataset = [
@@ -27,11 +27,11 @@ const errorDataset = [
 test.each(errorDataset)('It should convert wrong SensCritique note (%s) to null', (sensCritiqueNote, expected) => {
   const sensCritiqueRating = new SensCritiqueRating({
     name: 'test',
-    redirect: '',
+    redirect: ''
   })
-  const noteflixScore = sensCritiqueRating.ratingInPercent(sensCritiqueNote)
+  const sensCritiqueExtensionScore = sensCritiqueRating.ratingInPercent(sensCritiqueNote)
 
-  expect(noteflixScore).toBe(expected)
+  expect(sensCritiqueExtensionScore).toBe(expected)
 })
 
 test('It should create instance with rights rating', () => {
@@ -49,7 +49,7 @@ test('It should render node with rights info', () => {
     rating: '8,5',
     redirect: 'https://google.fr',
     hashId: 'XXX',
-    name: 'test',
+    name: 'test'
   })
   const sensCritiqueRatingRendered = sensCritiqueRating.render()
 

__tests__/storage/Cache.test.ts

@@ -5,14 +5,14 @@ import Cache from '../../src/storage/Cache'
 test('It should save information in sessionStorage', () => {
   const cache = new Cache()
   cache.save({ rating: '10', name: 'Breaking bad' }, Service.SENSCRITIQUE)
-  const item = sessionStorage.getItem('noteflix_5c68903a7be732b58d17c41f5c4c24ff')
+  const item = sessionStorage.getItem('senscritique_extension_5c68903a7be732b58d17c41f5c4c24ff')
 
   expect(item).toBe('{"rating":"10","name":"Breaking bad","hashId":"5c68903a7be732b58d17c41f5c4c24ff"}')
 })
 
 test('It should fetch information in sessionStorage', () => {
   const cache = new Cache()
-  sessionStorage.setItem('noteflix_5c68903a7be732b58d17c41f5c4c24ff', '{"rating":"10","name":"Breaking bad","hashId":"5c68903a7be732b58d17c41f5c4c24ff"}')
+  sessionStorage.setItem('senscritique_extension_5c68903a7be732b58d17c41f5c4c24ff', '{"rating":"10","name":"Breaking bad","hashId":"5c68903a7be732b58d17c41f5c4c24ff"}')
 
   const item = cache.get('Breaking bad', Service.SENSCRITIQUE)
   expect(item).toStrictEqual({ rating: '10', name: 'Breaking bad', hashId: '5c68903a7be732b58d17c41f5c4c24ff' })

manifest_chrome.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 2,
   "name": "SensCritique",
-  "version": "1.2.14",
+  "version": "1.2.15",
   "description": "Permet d'avoir les scores et avis SensCritique sur Netflix",
   "icons": {
     "48": "images/logo-48.png",

manifest_firefox.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 2,
   "name": "SensCritique",
-  "version": "1.2.14",
+  "version": "1.2.15",
   "description": "Permet d'avoir les scores et avis SensCritique sur Netflix",
   "icons": {
     "48": "images/logo-48.png",

package.json

@@ -1,10 +1,10 @@
 {
-  "name": "noteflix",
-  "version": "1.2.14",
+  "name": "senscritique-extension",
+  "version": "1.2.15",
   "description": "Permet d'avoir les scores SensCritique sur Netflix",
   "author": "SensCritique",
   "repository": {
-    "url": "https://github.com/senscritique/noteflix"
+    "url": "https://github.com/senscritique/SensCritique-extension"
   },
   "scripts": {
     "test": "jest"

src/dom/Manager.ts

@@ -111,7 +111,7 @@ export default class Manager {
   }
 
   showAbTestModal (): void {
-    const cacheKey = 'noteflix_help'
+    const cacheKey = 'senscritique_extension_help'
     const helpModalAlreadyDisplayed = sessionStorage.getItem(cacheKey)
 
     if (document.getElementById(AbTestModalId) == null && !helpModalAlreadyDisplayed) {
@@ -126,7 +126,7 @@ export default class Manager {
   }
 
   showNotSupportedModal (): void {
-    const cacheKey = 'noteflix_not_supported'
+    const cacheKey = 'senscritique_extension_not_supported'
     const NotSupportedModalIdDisplayed = sessionStorage.getItem(cacheKey)
 
     if (document.getElementById(NotSupportedModalId) == null && !NotSupportedModalIdDisplayed) {

src/dom/Modals.ts

@@ -1,7 +1,7 @@
 import logo64 from '../../images/logo'
 
-export const AbTestModalId = 'noteflix_help_modal'
-export const NotSupportedModalId = 'noteflix_not_supported_modal'
+export const AbTestModalId = 'senscritique_extension_help_modal'
+export const NotSupportedModalId = 'senscritique_extension_not_supported_modal'
 
 const Modal = (id: string, text: Element): Element => {
   const modal = document.createElement('div')
@@ -42,8 +42,8 @@ export const ABTestModal = (): Element => {
   button.innerText = 'Désactiver l\'option "Participer à des tests"'
   button.href = 'https://www.netflix.com/DoNotTest'
   text.innerHTML = `
-  Votre compte Netflix semble faire parti d'une phase de test, ce qui le rend incompatible avec NoteFlix.<br />
-  ${button.outerHTML} pour que NoteFlix fonctionne. 
+  Votre compte Netflix semble faire parti d'une phase de test, ce qui le rend incompatible avec SensCritique-extension.<br />
+  ${button.outerHTML} pour que SensCritique-extension fonctionne. 
   `
   return Modal(AbTestModalId, text)
 }
@@ -55,7 +55,7 @@ export const NotSupportedModal = (): Element => {
   text.style.marginLeft = '10px'
 
   text.innerHTML = `
-  L'interface de Netflix ne semble plus être supportée par Noteflix pour le moment. <br />
+  L'interface de Netflix ne semble plus être supportée par SensCritique-extension pour le moment. <br />
   Aucune crainte, nous sommes déjà sur le coup...🚀 <br /> 
   `
   return Modal(NotSupportedModalId, text)

src/dom/Ratings.ts

@@ -12,7 +12,7 @@ export default class Ratings {
     const senscritiqueElement = document.createElement('div')
     senscritiqueElement.style.padding = '0 .5em 0 .5em'
     senscritiqueElement.style.position = 'relative'
-    senscritiqueElement.classList.add(`senscritique_${hash}`)
+    senscritiqueElement.classList.add(`senscritique_extension_${hash}`)
     mainDiv.appendChild(senscritiqueElement)
 
     return mainDiv

src/http/SensCritique.ts

@@ -36,7 +36,7 @@ export default class SensCritique implements Client {
   async getVideoInfo (search: string, type: VideoType, year: string = null): Promise<VideoInfo> {
     if (search) {
       const headers = new Headers()
-      headers.append('User-Agent', `Noteflix v${app.version}`)
+      headers.append('User-Agent', `senscritique-extension v${app.version}`)
       headers.append('Content-Type', 'application/json')
       const response = await fetch(this.searchUrl, {
         headers,

src/logging/Logger.ts

@@ -17,7 +17,7 @@ export default class Logger {
       context: {
         app_version: chrome.runtime.getManifest().version,
         netflix_ui_version: Netflix.getUiVersion(),
-        service: 'noteflix'
+        service: 'senscritique-extension'
       }
     })
   }

src/storage/Cache.ts

@@ -6,7 +6,7 @@ export default class Cache {
   private readonly prefix: string;
 
   constructor () {
-    this.prefix = 'noteflix_'
+    this.prefix = 'senscritique_extension_'
   }
 
   save (videoInfo: VideoInfo, service: Service): VideoInfo {