Привет!

Може да погледнеш следните неща:

1. void List::addAll(const List& other){...}

Тук се иска да копираш стойностите от едини лист в другия, както е направено в момента ти правиш единия лист да ти е равен на другия по референция. Трябва да вземеш стойностите(values) една по една от other и да ги добавиш в настоящия лист, в него може да има стойности, като те се запазват.

2.  void List::removeFirst()

Предлагам ти следният вариант, макар че в случая и така рабати, то първо нищо не правиш по отношение на tail пойнтъра, ако имаш 0 елемента трябва да ти сочи към nullptr, другото е, че в момента не си сетнал предишния елемент на head да ти е nullptr, а сочи към освободена памет. :

void List::removeFirst() {
    if (size == 1) {
        delete this->head;
        this->head = nullptr;
        this->tail = nullptr;
        this->size--;
    } else if (size > 1) {
        Node *toDelete = this->head;
        this->head = this->head->getNext();
        this->head->setPrev(nullptr);
        delete toDelete;
        this->size--;
    }
}

3. List& List::operator=(const List& other)

Тук не освобождаваш паметта на настоящият обект, ако преди това е била заделена такава. Това си е memory leak

Поздрави!