Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds Unit Tests to Outlook #1707

Merged
merged 35 commits into from
Jun 5, 2016
Merged

Conversation

ThunderFrame
Copy link
Member

@ckuhn203 This one's for you.

Still need to RTF-escape content values
Temporary Fix in InspectionResultBase, for instances of unsaved filename
boom.
Adds ColumnInfos helper classes, for specifying and formatting column
titles, not yet fully implemented.
Adds a DocumentName property to QualifiedModuleName, adds DocumentName
to column output.
not implemented - need to write to events in the ThisOutlookSession
before this will work.
changes to Column alignment, font weights, colspans
Refines DocumentName of QualifiedModule
ProjectDisplayName eg. `Book1`
ProjectTitle eg. `VBA Project (Book1)
ComponentDisplayName eg. `Home`
ComponentTitle eg. `shtHome (Home)`
still needs testing across multiple hosts.
was out of Sync with Next, so had to add these fields to the newly
discovered ctor.
Adds Start and End Times for Unit Test results (only viewable in
Clipboard exports)
Title and Headings aren't yet localized.
No plain-text clipboard export format yet
Adds extra method for projects that have documents without a
document-component (eg. PowerPoint)
@rubberduck203
Copy link
Member

Beautiful hack! Just a couple of things. Why the empty catch? Mind adding some comments explaining why we need the hack to begin with? The current team know, but future maintainers might not immediately understand that Outlook doesn't have an Application.Run() method.

@retailcoder retailcoder merged commit ccd6225 into rubberduck-vba:next Jun 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants