phanan/koel

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  async fetch () {
    this.state.songs = songStore.syncWithVault(await http.get<Song[]>('songs/favorite'))
    return this.state.songs
  }

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

    cy.get('#extra').within(() => {
      cy.get('#extraTabYouTube').click()
      cy.get('[data-testid=youtube-search-result]:nth-child(2)').click()
    })

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      await waitFor(() => {
        expect(fetchMock).toHaveBeenCalledWith(folder)
        expect(queueMock).toHaveBeenCalledWith(songs, true)
        expect(goMock).toHaveBeenCalledWith('queue')
      })

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    it('does not have an option to download if downloading is disabled', async () => {
      commonStore.state.allow_download = false
      this.renderComponent()

      expect(screen.queryByText('Download')).toBeNull()

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    it('does not have an option to download if downloading is disabled', async () => {
      commonStore.state.allow_download = false
      this.renderComponent()

      expect(screen.queryByText('Download')).toBeNull()

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  async fetch () {
    this.state.songs = songStore.syncWithVault(await http.get<Song[]>('songs/recently-played'))
    return this.state.songs
  },

• Read up
  Read up

Function test has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  protected test () {
    it('downloads', async () => {
      const downloadMock = this.mock(downloadService, 'fromAlbum')
      await this.renderComponent()

Function test has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  protected test () {
    it('renders an empty state if no songs found', async () => {
      commonStore.state.song_length = 0
      this.mock(overviewStore, 'init')

Function test has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  protected test () {
    it('functions', async () => {
      const song = factory<Song>('song')

      const searchMock = this.mock(youTubeService, 'searchVideosBySong').mockResolvedValueOnce({

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      await waitFor(() => {
        expect(fetchMock).toHaveBeenCalledWith(playlist)
        expect(queueMock).toHaveBeenCalledWith(songs)
        expect(goMock).toHaveBeenCalledWith('queue')
      })

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      await waitFor(() => {
        expect(fetchMock).toHaveBeenCalledWith(folder)
        expect(queueMock).toHaveBeenCalledWith(songs)
        expect(goMock).toHaveBeenCalledWith('queue')
      })

• Read up
  Read up

Function initScene has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  initScene () {
    const _sphere_size = .7
    const _geom = new THREE.SphereGeometry(_sphere_size, 128, 128)
    const _geom_lowres = new THREE.SphereGeometry(_sphere_size, 64, 64)

Function test has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  protected test () {
    it('downloads songs', () => {
      const mock = this.mock(downloadService, 'trigger')
      downloadService.fromSongs([factory<Song>('song', { id: 'foo' }), factory<Song>('song', { id: 'bar' })])

Function update has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  update () {
    this.analyzer.getByteFrequencyData(this.audioBuffer)
    let bass = 0., mid = 0., high = 0.

    if (this.audioBuffer[0] === 0) {

Function constructor has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  constructor () {
    this.client = Axios.create({
      baseURL: `${window.BASE_URL}api`,
      headers: {
        'X-Api-Version': 'v6'

Identical blocks of code found in 2 locations. Consider refactoring.

Open

      await waitFor(() => {
        expect(fetchMock).toHaveBeenCalled()
        expect(playMock).toHaveBeenCalledWith(songs)
        expect(goMock).toHaveBeenCalledWith('queue')
      })

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

      await waitFor(() => {
        expect(fetchMock).toHaveBeenCalled()
        expect(playMock).toHaveBeenCalledWith(songs)
        expect(goMock).toHaveBeenCalledWith('queue')
      })

• Read up
  Read up

Method make has 9 arguments (exceeds 4 allowed). Consider refactoring.

Open

        ?string $title,
        ?string $artistName,
        ?string $albumName,
        ?string $albumArtistName,
        ?int $track,

Method __construct has 9 arguments (exceeds 4 allowed). Consider refactoring.

Open

        public ?string $title,
        public ?string $artistName,
        public ?string $albumName,
        public ?string $albumArtistName,
        public ?int $track,

Function test has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  protected test () {
    it('renders the queue', () => {
      this.renderComponent(factory<Song>('song', 3))

      expect(screen.queryByTestId('song-list')).toBeTruthy()