mirror of
https://github.com/tgstation/tgstation-server.git
synced 2026-08-26 22:48:20 +01:00
Sanely namespaces the project
This commit is contained in:
Cyberboss
@@ -0,0 +1,372 @@
|
||||
using Octokit;
|
||||
using System;
|
||||
using System.Collections.Generic;
|
||||
using System.Linq;
|
||||
using System.Threading.Tasks;
|
||||
using System.Windows.Forms;
|
||||
using TGS.Interface;
|
||||
using TGS.Interface.Components;
|
||||
|
||||
namespace TGS.ControlPanel
|
||||
{
|
||||
sealed partial class TestMergeManager : ServerOpForm
|
||||
{
|
||||
/// <summary>
|
||||
/// Error message format used when <see cref="ITGRepository.MergedPullRequests(out string)"/> fails
|
||||
/// </summary>
|
||||
const string MergedPullsError = "Error retrieving currently merged pull requests: {0}";
|
||||
|
||||
/// <summary>
|
||||
/// The <see cref="IServerInterface"/> connected to an <see cref="ITGInstance"/> to handle the pull requests for
|
||||
/// </summary>
|
||||
readonly IServerInterface currentInterface;
|
||||
|
||||
/// <summary>
|
||||
/// The <see cref="GitHubClient"/> to use to read PR lists
|
||||
/// </summary>
|
||||
readonly GitHubClient client;
|
||||
|
||||
/// <summary>
|
||||
/// The owner of the target <see cref="Repository"/>
|
||||
/// </summary>
|
||||
string repoOwner;
|
||||
/// <summary>
|
||||
/// The name of the target <see cref="Repository"/>
|
||||
/// </summary>
|
||||
string repoName;
|
||||
|
||||
/// <summary>
|
||||
/// Construct a <see cref="TestMergeManager"/>
|
||||
/// </summary>
|
||||
/// <param name="interfaceToUse">The <see cref="IServerInterface"/> to use for managing the <see cref="ITGInstance"/></param>
|
||||
/// <param name="clientToUse">The <see cref="GitHubClient"/> to use for getting pull request information</param>
|
||||
public TestMergeManager(IServerInterface interfaceToUse, GitHubClient clientToUse)
|
||||
{
|
||||
InitializeComponent();
|
||||
DialogResult = DialogResult.Cancel;
|
||||
UpdateToRemoteRadioButton.Checked = true;
|
||||
currentInterface = interfaceToUse;
|
||||
client = clientToUse;
|
||||
Load += PullRequestManager_Load;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Populate <see cref="PullRequestListBox"/> with <see cref="PullRequestInfo"/> from github and check off ones that are currently merged. Prompts the user to login to GitHub if they hit the rate limit
|
||||
/// </summary>
|
||||
async void LoadPullRequests()
|
||||
{
|
||||
try
|
||||
{
|
||||
Enabled = false;
|
||||
UseWaitCursor = true;
|
||||
try
|
||||
{
|
||||
PullRequestListBox.Items.Clear();
|
||||
|
||||
var repo = currentInterface.GetComponent<ITGRepository>();
|
||||
string error = null;
|
||||
List<PullRequestInfo> pulls = null;
|
||||
|
||||
//get started on this while we're processing here
|
||||
var pullsRequest = Task.Factory.StartNew(() => pulls = repo.MergedPullRequests(out error));
|
||||
|
||||
//Search for open PRs
|
||||
Enabled = false;
|
||||
UseWaitCursor = true;
|
||||
SearchIssuesResult result;
|
||||
try
|
||||
{
|
||||
result = await client.Search.SearchIssues(new SearchIssuesRequest
|
||||
{
|
||||
Repos = new RepositoryCollection { { repoOwner, repoName } },
|
||||
State = ItemState.Open,
|
||||
Type = IssueTypeQualifier.PullRequest
|
||||
});
|
||||
}
|
||||
finally
|
||||
{
|
||||
Enabled = true;
|
||||
UseWaitCursor = false;
|
||||
}
|
||||
|
||||
//now we need to know what's merged
|
||||
await pullsRequest;
|
||||
if (pulls == null)
|
||||
MessageBox.Show(String.Format(MergedPullsError, error));
|
||||
|
||||
//insert the open pull requests, checking already merged once
|
||||
foreach (var I in result.Items)
|
||||
{
|
||||
bool alreadyMerged = false;
|
||||
var pull = pulls.Where(x => x.Number == I.Number).FirstOrDefault();
|
||||
if (pull != null)
|
||||
{
|
||||
//we need the full info for this PR
|
||||
alreadyMerged = (await client.PullRequest.Get(repoOwner, repoName, I.Number)).Head.Sha == pull.Sha;
|
||||
if (alreadyMerged)
|
||||
pulls.Remove(pull);
|
||||
}
|
||||
InsertPullRequest(I, false, alreadyMerged);
|
||||
}
|
||||
|
||||
//insert remaining merged pulls
|
||||
foreach (var I in pulls)
|
||||
InsertItem(String.Format("#{0} - {1} - OUTDATED: {2}", I.Number, I.Title, I.Sha), true, true);
|
||||
}
|
||||
finally
|
||||
{
|
||||
Enabled = true;
|
||||
UseWaitCursor = false;
|
||||
}
|
||||
}
|
||||
catch (ForbiddenException)
|
||||
{
|
||||
if (client.Credentials.AuthenticationType == AuthenticationType.Anonymous) //assume request limit hit
|
||||
{
|
||||
if(Program.RateLimitPrompt(client))
|
||||
LoadPullRequests();
|
||||
}
|
||||
else
|
||||
throw;
|
||||
}
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Format an entry for <paramref name="issue"/> and insert it into <see cref="PullRequestListBox"/>
|
||||
/// </summary>
|
||||
/// <param name="issue">The <see cref="Issue"/> to format, must contain a <see cref="PullRequest"/></param>
|
||||
/// <param name="prioritize">If this or <paramref name="isChecked"/> is <see langword="true"/>, <paramref name="issue"/> will be inserted at the top of <see cref="PullRequestListBox"/> as opposed to the bottom</param>
|
||||
/// <param name="isChecked">If the item should be checked</param>
|
||||
void InsertPullRequest(Issue issue, bool prioritize, bool isChecked)
|
||||
{
|
||||
bool needsTesting = false;
|
||||
foreach (var J in issue.Labels)
|
||||
if (J.Name.ToLower().Contains("test"))
|
||||
{
|
||||
needsTesting = true;
|
||||
break;
|
||||
}
|
||||
var itemString = String.Format("#{0} - {1}{2}", issue.Number, issue.Title, issue.PullRequest != null && issue.PullRequest.Merged ? " - MERGED ON REMOTE" : needsTesting ? " - TESTING REQUESTED" : "");
|
||||
InsertItem(itemString, prioritize || needsTesting, isChecked);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Insert an <paramref name="itemString"/> into <see cref="PullRequestListBox"/>
|
||||
/// </summary>
|
||||
/// <param name="itemString">The <see cref="string"/> to insert</param>
|
||||
/// <param name="prioritize">If this or <paramref name="isChecked"/> is <see langword="true"/>, <paramref name="itemString"/> will be inserted at the top of <see cref="PullRequestListBox"/> as opposed to the bottom</param>
|
||||
/// <param name="isChecked">If the item should be checked</param>
|
||||
void InsertItem(string itemString, bool prioritize, bool isChecked)
|
||||
{
|
||||
prioritize = prioritize || isChecked;
|
||||
if (prioritize)
|
||||
{
|
||||
PullRequestListBox.Items.Insert(0, itemString);
|
||||
if (isChecked)
|
||||
PullRequestListBox.SetItemChecked(0, true);
|
||||
}
|
||||
else
|
||||
PullRequestListBox.Items.Add(itemString, isChecked);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Called when the <see cref="TestMergeManager"/> is loaded. Sets <see cref="repoOwner"/> and <see cref="repoName"/>
|
||||
/// </summary>
|
||||
/// <param name="sender">The sender of the event</param>
|
||||
/// <param name="e">The <see cref="EventArgs"/></param>
|
||||
void PullRequestManager_Load(object sender, EventArgs e)
|
||||
{
|
||||
Enabled = false;
|
||||
var repo = currentInterface.GetComponent<ITGRepository>();
|
||||
if(!Program.GetRepositoryRemote(repo, out repoOwner, out repoName))
|
||||
{
|
||||
Close();
|
||||
return;
|
||||
}
|
||||
LoadPullRequests();
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Calls <see cref="ITGRepository.GenerateChangelog(out string)"/> and shows the user an error prompt if it fails
|
||||
/// </summary>
|
||||
/// <param name="repo">The <see cref="ITGRepository"/> to call <see cref="ITGRepository.GenerateChangelog(out string)"/> on</param>
|
||||
async void GenerateChangelog(ITGRepository repo)
|
||||
{
|
||||
string error = null;
|
||||
await WrapServerOp(() => repo.GenerateChangelog(out error));
|
||||
if (error != null)
|
||||
MessageBox.Show(String.Format("Error generating changelog: {0}", error));
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Called when the <see cref="ApplyButton"/> is clicked. Calls <see cref="ITGRepository.Update(bool)"/> if necessary, merge pull requests, and call <see cref="ITGCompiler.Compile(bool)"/>. Closes the <see cref="TestMergeManager"/> if appropriate
|
||||
/// </summary>
|
||||
/// <param name="sender">The sender of the event</param>
|
||||
/// <param name="e">The <see cref="EventArgs"/></param>
|
||||
async void ApplyButton_Click(object sender, EventArgs e)
|
||||
{
|
||||
Enabled = false;
|
||||
UseWaitCursor = true;
|
||||
ApplyingPullRequestsLabel.Visible = true;
|
||||
ApplyingPullRequestsProgressBar.Visible = true;
|
||||
PullRequestListBox.Visible = false;
|
||||
try
|
||||
{
|
||||
//so first collect a list of pulls that are checked
|
||||
var pulls = new Dictionary<int, string>();
|
||||
foreach (var I in PullRequestListBox.CheckedItems)
|
||||
{
|
||||
var S = (string)I;
|
||||
string mergedSha = null;
|
||||
var splits = S.Split(' ');
|
||||
if(S.Contains(" - OUTDATED: "))
|
||||
mergedSha = splits[splits.Length - 1];
|
||||
var key = Convert.ToInt32((splits[0].Substring(1)));
|
||||
try
|
||||
{
|
||||
pulls.Add(key, mergedSha);
|
||||
}
|
||||
catch
|
||||
{
|
||||
MessageBox.Show(String.Format("Checked both keep and update option for #{0}", key), "Error");
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
var repo = currentInterface.GetComponent<ITGRepository>();
|
||||
|
||||
string error = null;
|
||||
//Do standard repo updates
|
||||
if (UpdateToRemoteRadioButton.Checked)
|
||||
await WrapServerOp(() => error = repo.Update(true));
|
||||
else if (UpdateToOriginRadioButton.Checked)
|
||||
await WrapServerOp(() => error = repo.Reset(true));
|
||||
|
||||
if (error != null)
|
||||
{
|
||||
MessageBox.Show(String.Format("Error updating repository: {0}", error));
|
||||
return;
|
||||
}
|
||||
|
||||
if (UpdateToRemoteRadioButton.Checked)
|
||||
{
|
||||
GenerateChangelog(repo);
|
||||
await WrapServerOp(() => error = repo.SynchronizePush());
|
||||
if (error != null)
|
||||
MessageBox.Show(String.Format("Error sychronizing repo: {0}", error));
|
||||
}
|
||||
|
||||
//Merge the PRs, collect errors
|
||||
IList<string> errors = null;
|
||||
await WrapServerOp(() =>
|
||||
{
|
||||
errors = new List<string>();
|
||||
foreach (var I in pulls)
|
||||
{
|
||||
var res = repo.MergePullRequest(I.Key, I.Value);
|
||||
if (res != null)
|
||||
errors.Add(String.Format("Error merging PR #{0}: {1}", I, res));
|
||||
}
|
||||
});
|
||||
|
||||
//Show any errors
|
||||
foreach (var I in errors)
|
||||
MessageBox.Show(I);
|
||||
if (errors.Count != 0)
|
||||
return;
|
||||
|
||||
if (pulls.Count > 0)
|
||||
//regen the changelog
|
||||
GenerateChangelog(repo);
|
||||
|
||||
//Start the compile
|
||||
if (!currentInterface.GetComponent<ITGCompiler>().Compile(pulls.Count == 1))
|
||||
MessageBox.Show("Could not start compilation!");
|
||||
else
|
||||
MessageBox.Show("Test merges updated and compilation started!");
|
||||
}
|
||||
finally
|
||||
{
|
||||
Enabled = true;
|
||||
UseWaitCursor = false;
|
||||
ApplyingPullRequestsLabel.Visible = false;
|
||||
ApplyingPullRequestsProgressBar.Visible = false;
|
||||
PullRequestListBox.Visible = true;
|
||||
}
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Called when the <see cref="RefreshButton"/> is clicked
|
||||
/// </summary>
|
||||
/// <param name="sender">The sender of the event</param>
|
||||
/// <param name="e">The <see cref="EventArgs"/></param>
|
||||
void RefreshButton_Click(object sender, EventArgs e)
|
||||
{
|
||||
LoadPullRequests();
|
||||
}
|
||||
|
||||
bool PullRequestsStringsListContainsPRNumber(CheckedListBox.ObjectCollection items, int PRNumber)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Called when the <see cref="AddPRButton"/> is clicked. Adds the PR with the number in <see cref="AddPRNumericUpDown"/> to <see cref="PullRequestListBox"/> or shows an error prompt if it doesn't/already exists
|
||||
/// </summary>
|
||||
/// <param name="sender">The sender of the event</param>
|
||||
/// <param name="e">The <see cref="EventArgs"/></param>
|
||||
async void AddPRButton_Click(object sender, EventArgs e)
|
||||
{
|
||||
|
||||
Enabled = false;
|
||||
UseWaitCursor = true;
|
||||
try
|
||||
{
|
||||
IList<PullRequestInfo> pulls = null;
|
||||
string error = null;
|
||||
var mergedPullsTask = WrapServerOp(() => pulls = currentInterface.GetComponent<ITGRepository>().MergedPullRequests(out error));
|
||||
|
||||
int PRNumber;
|
||||
try
|
||||
{
|
||||
PRNumber = Convert.ToInt32(AddPRNumericUpDown.Value);
|
||||
}
|
||||
catch
|
||||
{
|
||||
MessageBox.Show("Invalid PR number!");
|
||||
return;
|
||||
}
|
||||
var found = false;
|
||||
var asString = PRNumber.ToString();
|
||||
foreach (var I in PullRequestListBox.Items)
|
||||
if (((string)I).Split(' ')[0].Substring(1) == asString)
|
||||
{
|
||||
found = true;
|
||||
break;
|
||||
}
|
||||
if (found)
|
||||
{
|
||||
MessageBox.Show("That PR is already in the list!");
|
||||
return;
|
||||
}
|
||||
|
||||
await mergedPullsTask;
|
||||
if (pulls == null)
|
||||
MessageBox.Show(String.Format(MergedPullsError, error));
|
||||
//get the PR in question
|
||||
var PR = await client.Issue.Get(repoName, repoOwner, PRNumber);
|
||||
if(PR == null ||PR.PullRequest == null)
|
||||
{
|
||||
MessageBox.Show("That doesn't seem to be a valid PR!");
|
||||
return;
|
||||
}
|
||||
InsertPullRequest(PR, true, pulls == null || pulls.Any(x => x.Number == PRNumber));
|
||||
}
|
||||
finally
|
||||
{
|
||||
UseWaitCursor = false;
|
||||
Enabled = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user