from validate_docbr import CNPJ
cliente = "60746948000112"
cnpj = CNPJ()
cadastro_aprovado = cnpj.validate(cliente)
if cadastro_aprovado == True:
print("o CNPJ {} é válido.".format(cnpj.mask(cliente)))
else:
print("O CNPJ {} não é inválido".format(cnpj.mask(cliente)))
Errado em que sentido? Em termos de comportamento parece certo, ele vai imprimir se o numero é um cnpj válido.
Mas em termos de design, é um tipo de solução que não é muito organizada e reutilizável. Se vc (ou outra pessoa) for usar essa solução em outro lugar, terá que copiar e colar o codigo. Se em algum momento vc mudar alguma coisa nele, ou quiser adicionar uma funcionalidade, terá que modificar esse codigo em todos os lugares que vc o utiliza.
Se é um script só pra vc, uma coisa que vc quer fazer rápido e não vai ser reutilizada, acho que é valido uma abordagem assim. Mas se vc pretende se preparar para trabalhar em projetos normais, utilizando boas práticas, vc deveria seguir a recomendação do curso de implementar um design orientado a objetos, encapsulando a funcionalidade em uma classe Validadora e utilizando instancias dela para validar o que vc precisar.