Re-enable bitbucket test

[jayfk]

Bitbucket repo tests are currently disabled (4ac30c5).

[vaibhav-jain]

@jayfk Do you recall why it was disabled? Please share if you do :)

[dmoney]

I uncommented it, fixed the compile errors ( branch, 032c8b6 ), and tried running the tests, but the two tests fail because the django bitbucket repo has been moved to github.

Looks like all the tests in test_repos.py are commented out or not there, so I'm trying to do the same thing with the GitHub tests to see if they'll work. (I think I need to set up a GitHub API key for that).

In general, should tests depend on a specific repo? Django could change their repo's tagline or move the repo to GitBlarg and these tests would start failing (if they were currently passing). Maybe the git hosts' APIs have a test repo that's guaranteed to stay there. For github we could probably use djangopackages' own repo.

[dmoney]

Okay, github tests (3397390) passed once I created a personal access token and set GITHUB_TOKEN to that in .env.local.

[dmoney]

I found some repos that are still on Bitbucket, but it doesn't seem to be heavily used anymore.

Of the 20ish bitbucket-hosted packages listed in Django Packages' CMS grid, only 2 are still on Bitbucket (clicking the links from the grid gave a 404 for the others). And the most recent one's last commit was in 2014.

The tests are still failing because it's not getting anything back for these repos, but not throwing an error (but then it didn't when it used django bitbucket repo URL either). I'm guessing some API authentication is required.

[dmoney]

I updated the Bitbucket repo handler to use the Bitbucket Cloud 2.0 API (aa2ac2a), but one of the tests still fails because the repos have 0 forks. I'll see if I can find a more active one.

[dmoney]

Found a repo with forks and got the tests passing. I'll create a PR once I clean up the script that I used to find the repo, so it can be used again if that repo goes away.

Also maybe I'll add something about the GITHUB_TOKEN to the docs.

[dmoney]

Created PR #793.